[FORMATTING] Standardize win32csr to 4-space indents. Based on a patch by Adam Kachwa...
[reactos.git] / reactos / subsystems / win32 / csrss / win32csr / alias.c
index 8b3254b..569948a 100644 (file)
@@ -5,23 +5,22 @@
  * PURPOSE:     CSRSS alias support functions
  * COPYRIGHT:   Christoph Wittich
  *              Johannes Anderwald
- *               
+ *
  */
 
 
 /* INCLUDES ******************************************************************/
 
-#include "w32csr.h"
-
 #define NDEBUG
+#include "w32csr.h"
 #include <debug.h>
 
 typedef struct tagALIAS_ENTRY
 {
-   LPCWSTR lpSource;
-   LPCWSTR lpTarget;
-   struct tagALIAS_ENTRY * Next;
-}ALIAS_ENTRY, *PALIAS_ENTRY;
+    LPCWSTR lpSource;
+    LPCWSTR lpTarget;
+    struct tagALIAS_ENTRY * Next;
+} ALIAS_ENTRY, *PALIAS_ENTRY;
 
 
 typedef struct tagALIAS_HEADER
@@ -30,7 +29,7 @@ typedef struct tagALIAS_HEADER
     PALIAS_ENTRY Data;
     struct tagALIAS_HEADER * Next;
 
-}ALIAS_HEADER, *PALIAS_HEADER;
+} ALIAS_HEADER, *PALIAS_HEADER;
 
 /* Ensure that a buffer is contained within the process's shared memory section. */
 static BOOL
@@ -38,10 +37,10 @@ ValidateBuffer(PCSRSS_PROCESS_DATA ProcessData, PVOID Buffer, ULONG Size)
 {
     ULONG Offset = (BYTE *)Buffer - (BYTE *)ProcessData->CsrSectionViewBase;
     if (Offset >= ProcessData->CsrSectionViewSize
-        || Size > (ProcessData->CsrSectionViewSize - Offset))
+            || Size > (ProcessData->CsrSectionViewSize - Offset))
     {
         DPRINT1("Invalid buffer %p %d; not within %p %d\n",
-            Buffer, Size, ProcessData->CsrSectionViewBase, ProcessData->CsrSectionViewSize);
+                Buffer, Size, ProcessData->CsrSectionViewBase, ProcessData->CsrSectionViewSize);
         return FALSE;
     }
     return TRUE;
@@ -68,51 +67,54 @@ IntFindAliasHeader(PALIAS_HEADER RootHeader, LPCWSTR lpExeName)
 PALIAS_HEADER
 IntCreateAliasHeader(LPCWSTR lpExeName)
 {
-  PALIAS_HEADER Entry;
-  UINT dwLength = wcslen(lpExeName) + 1;
-
-  Entry = RtlAllocateHeap(Win32CsrApiHeap, 0, sizeof(ALIAS_HEADER) + sizeof(WCHAR) * dwLength);
-  if (!Entry)
-      return Entry;
-  
-  Entry->lpExeName = (LPCWSTR)(Entry + 1);
-  wcscpy((WCHAR*)Entry->lpExeName, lpExeName);
-  Entry->Data = NULL;
-  Entry->Next = NULL;
-  return Entry;
+    PALIAS_HEADER Entry;
+    UINT dwLength = wcslen(lpExeName) + 1;
+
+    Entry = RtlAllocateHeap(Win32CsrApiHeap, 0, sizeof(ALIAS_HEADER) + sizeof(WCHAR) * dwLength);
+    if (!Entry)
+        return Entry;
+
+    Entry->lpExeName = (LPCWSTR)(Entry + 1);
+    wcscpy((WCHAR*)Entry->lpExeName, lpExeName);
+    Entry->Data = NULL;
+    Entry->Next = NULL;
+    return Entry;
 }
 
 VOID
 IntInsertAliasHeader(PALIAS_HEADER * RootHeader, PALIAS_HEADER NewHeader)
 {
-  PALIAS_HEADER CurrentHeader;
-  PALIAS_HEADER *LastLink = RootHeader;
-
-  while ((CurrentHeader = *LastLink) != NULL)
-  {
-      INT Diff = _wcsicmp(NewHeader->lpExeName, CurrentHeader->lpExeName);
-      if (Diff < 0)
-      {
-        break;
-      }
-      LastLink = &CurrentHeader->Next;
-  }
-
-  *LastLink = NewHeader;
-  NewHeader->Next = CurrentHeader;
+    PALIAS_HEADER CurrentHeader;
+    PALIAS_HEADER *LastLink = RootHeader;
+
+    while ((CurrentHeader = *LastLink) != NULL)
+    {
+        INT Diff = _wcsicmp(NewHeader->lpExeName, CurrentHeader->lpExeName);
+        if (Diff < 0)
+        {
+            break;
+        }
+        LastLink = &CurrentHeader->Next;
+    }
+
+    *LastLink = NewHeader;
+    NewHeader->Next = CurrentHeader;
 }
 
 PALIAS_ENTRY
 IntGetAliasEntry(PALIAS_HEADER Header, LPCWSTR lpSrcName)
 {
+    PALIAS_ENTRY RootHeader;
+
     if (Header == NULL)
         return NULL;
 
-    PALIAS_ENTRY RootHeader = Header->Data;
+    RootHeader = Header->Data;
     while(RootHeader)
     {
+        INT diff;
         DPRINT("IntGetAliasEntry>lpSource %S\n", RootHeader->lpSource);
-        INT diff = _wcsicmp(RootHeader->lpSource, lpSrcName);
+        diff = _wcsicmp(RootHeader->lpSource, lpSrcName);
         if (!diff)
             return RootHeader;
 
@@ -128,60 +130,60 @@ IntGetAliasEntry(PALIAS_HEADER Header, LPCWSTR lpSrcName)
 VOID
 IntInsertAliasEntry(PALIAS_HEADER Header, PALIAS_ENTRY NewEntry)
 {
-  PALIAS_ENTRY CurrentEntry;
-  PALIAS_ENTRY *LastLink = &Header->Data;
+    PALIAS_ENTRY CurrentEntry;
+    PALIAS_ENTRY *LastLink = &Header->Data;
 
-  while ((CurrentEntry = *LastLink) != NULL)
-  {
-    INT Diff = _wcsicmp(NewEntry->lpSource, CurrentEntry->lpSource);
-    if (Diff < 0)
+    while ((CurrentEntry = *LastLink) != NULL)
     {
-        break;
+        INT Diff = _wcsicmp(NewEntry->lpSource, CurrentEntry->lpSource);
+        if (Diff < 0)
+        {
+            break;
+        }
+        LastLink = &CurrentEntry->Next;
     }
-    LastLink = &CurrentEntry->Next;
-  }
 
-  *LastLink = NewEntry;
-  NewEntry->Next = CurrentEntry;
+    *LastLink = NewEntry;
+    NewEntry->Next = CurrentEntry;
 }
 
 PALIAS_ENTRY
 IntCreateAliasEntry(LPCWSTR lpSource, LPCWSTR lpTarget)
 {
-   UINT dwSource;
-   UINT dwTarget;
-   PALIAS_ENTRY Entry;
+    UINT dwSource;
+    UINT dwTarget;
+    PALIAS_ENTRY Entry;
 
-   dwSource = wcslen(lpSource) + 1;
-   dwTarget = wcslen(lpTarget) + 1;
+    dwSource = wcslen(lpSource) + 1;
+    dwTarget = wcslen(lpTarget) + 1;
 
-   Entry = RtlAllocateHeap(Win32CsrApiHeap, 0, sizeof(ALIAS_ENTRY) + sizeof(WCHAR) * (dwSource + dwTarget));
-   if (!Entry)
-       return Entry;
+    Entry = RtlAllocateHeap(Win32CsrApiHeap, 0, sizeof(ALIAS_ENTRY) + sizeof(WCHAR) * (dwSource + dwTarget));
+    if (!Entry)
+        return Entry;
 
-   Entry->lpSource = (LPCWSTR)(Entry + 1);
-   wcscpy((LPWSTR)Entry->lpSource, lpSource);
-   Entry->lpTarget = Entry->lpSource + dwSource;
-   wcscpy((LPWSTR)Entry->lpTarget, lpTarget);
-   Entry->Next = NULL;
+    Entry->lpSource = (LPCWSTR)(Entry + 1);
+    wcscpy((LPWSTR)Entry->lpSource, lpSource);
+    Entry->lpTarget = Entry->lpSource + dwSource;
+    wcscpy((LPWSTR)Entry->lpTarget, lpTarget);
+    Entry->Next = NULL;
 
-   return Entry;
+    return Entry;
 }
 
 UINT
 IntGetConsoleAliasesExesLength(PALIAS_HEADER RootHeader)
 {
-   UINT length = 0;
+    UINT length = 0;
 
-   while(RootHeader)
-   {
-       length += (wcslen(RootHeader->lpExeName) + 1) * sizeof(WCHAR);
-       RootHeader = RootHeader->Next;
-   }
-   if (length)
-       length += sizeof(WCHAR); // last entry entry is terminated with 2 zero bytes
+    while(RootHeader)
+    {
+        length += (wcslen(RootHeader->lpExeName) + 1) * sizeof(WCHAR);
+        RootHeader = RootHeader->Next;
+    }
+    if (length)
+        length += sizeof(WCHAR); // last entry entry is terminated with 2 zero bytes
 
-   return length;
+    return length;
 }
 
 UINT
@@ -213,22 +215,22 @@ IntGetConsoleAliasesExes(PALIAS_HEADER RootHeader, LPWSTR TargetBuffer, UINT Tar
 UINT
 IntGetAllConsoleAliasesLength(PALIAS_HEADER Header)
 {
-   UINT Length = 0;
-   PALIAS_ENTRY CurEntry = Header->Data;
-
-   while(CurEntry)
-   {
-       Length += wcslen(CurEntry->lpSource);
-       Length += wcslen(CurEntry->lpTarget);
-       Length += 2; // zero byte and '='
-       CurEntry = CurEntry->Next;
-   }
-
-   if (Length)
-   {
-       return (Length+1) * sizeof(WCHAR);
-   }
-   return 0;
+    UINT Length = 0;
+    PALIAS_ENTRY CurEntry = Header->Data;
+
+    while(CurEntry)
+    {
+        Length += wcslen(CurEntry->lpSource);
+        Length += wcslen(CurEntry->lpTarget);
+        Length += 2; // zero byte and '='
+        CurEntry = CurEntry->Next;
+    }
+
+    if (Length)
+    {
+        return (Length+1) * sizeof(WCHAR);
+    }
+    return 0;
 }
 UINT
 IntGetAllConsoleAliases(PALIAS_HEADER Header, LPWSTR TargetBuffer, UINT TargetBufferLength)
@@ -312,10 +314,9 @@ CSR_API(CsrAddConsoleAlias)
 
     if (lpExeName == NULL || lpSource == NULL)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
-    
+
     Request->Status = ConioConsoleFromProcessData(ProcessData, &Console);
     if (!NT_SUCCESS(Request->Status))
     {
@@ -328,9 +329,8 @@ CSR_API(CsrAddConsoleAlias)
         Header = IntCreateAliasHeader(lpExeName);
         if (!Header)
         {
-            Request->Status = STATUS_INSUFFICIENT_RESOURCES;
             ConioUnlockConsole(Console);
-            return Request->Status;
+            return STATUS_INSUFFICIENT_RESOURCES;
         }
         IntInsertAliasHeader(&Console->Aliases, Header);
     }
@@ -355,15 +355,13 @@ CSR_API(CsrAddConsoleAlias)
 
     if (!Entry)
     {
-        Request->Status = STATUS_INSUFFICIENT_RESOURCES;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INSUFFICIENT_RESOURCES;
     }
 
     IntInsertAliasEntry(Header, Entry);
-    Request->Status = STATUS_SUCCESS;
     ConioUnlockConsole(Console);
-    return Request->Status;
+    return STATUS_SUCCESS;
 }
 
 CSR_API(CsrGetConsoleAlias)
@@ -381,14 +379,13 @@ CSR_API(CsrGetConsoleAlias)
     lpTarget = Request->Data.GetConsoleAlias.TargetBuffer;
 
 
-    DPRINT("CsrGetConsoleAlias entered lpExeName %p lpSource %p TargetBuffer %p TargetBufferLength %u\n", 
-        lpExeName, lpSource, lpTarget, Request->Data.GetConsoleAlias.TargetBufferLength);
-    
-    if (Request->Data.GetConsoleAlias.ExeLength == 0 || lpTarget == NULL || 
-        Request->Data.GetConsoleAlias.TargetBufferLength == 0 || Request->Data.GetConsoleAlias.SourceLength == 0)
+    DPRINT("CsrGetConsoleAlias entered lpExeName %p lpSource %p TargetBuffer %p TargetBufferLength %u\n",
+           lpExeName, lpSource, lpTarget, Request->Data.GetConsoleAlias.TargetBufferLength);
+
+    if (Request->Data.GetConsoleAlias.ExeLength == 0 || lpTarget == NULL ||
+            Request->Data.GetConsoleAlias.TargetBufferLength == 0 || Request->Data.GetConsoleAlias.SourceLength == 0)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Request->Status = ConioConsoleFromProcessData(ProcessData, &Console);
@@ -400,39 +397,34 @@ CSR_API(CsrGetConsoleAlias)
     Header = IntFindAliasHeader(Console->Aliases, lpExeName);
     if (!Header)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Entry = IntGetAliasEntry(Header, lpSource);
     if (!Entry)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Length = (wcslen(Entry->lpTarget)+1) * sizeof(WCHAR);
     if (Length > Request->Data.GetConsoleAlias.TargetBufferLength)
     {
-        Request->Status = STATUS_BUFFER_TOO_SMALL;
         ConioUnlockConsole(Console);
-        return Request->Status;      
+        return STATUS_BUFFER_TOO_SMALL;
     }
 
     if (!ValidateBuffer(ProcessData, lpTarget, Request->Data.GetConsoleAlias.TargetBufferLength))
     {
-        Request->Status = STATUS_ACCESS_VIOLATION;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_ACCESS_VIOLATION;
     }
 
     wcscpy(lpTarget, Entry->lpTarget);
     Request->Data.GetConsoleAlias.BytesWritten = Length;
-    Request->Status = STATUS_SUCCESS;
     ConioUnlockConsole(Console);
-    return Request->Status;
+    return STATUS_SUCCESS;
 }
 
 CSR_API(CsrGetAllConsoleAliases)
@@ -443,8 +435,7 @@ CSR_API(CsrGetAllConsoleAliases)
 
     if (Request->Data.GetAllConsoleAlias.lpExeName == NULL)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Request->Status = ConioConsoleFromProcessData(ProcessData, &Console);
@@ -456,35 +447,31 @@ CSR_API(CsrGetAllConsoleAliases)
     Header = IntFindAliasHeader(Console->Aliases, Request->Data.GetAllConsoleAlias.lpExeName);
     if (!Header)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     if (IntGetAllConsoleAliasesLength(Header) > Request->Data.GetAllConsoleAlias.AliasBufferLength)
     {
-        Request->Status = STATUS_BUFFER_OVERFLOW;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_BUFFER_OVERFLOW;
     }
 
     if (!ValidateBuffer(ProcessData,
                         Request->Data.GetAllConsoleAlias.AliasBuffer,
                         Request->Data.GetAllConsoleAlias.AliasBufferLength))
     {
-        Request->Status = STATUS_ACCESS_VIOLATION;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_ACCESS_VIOLATION;
     }
 
-    BytesWritten = IntGetAllConsoleAliases(Header, 
+    BytesWritten = IntGetAllConsoleAliases(Header,
                                            Request->Data.GetAllConsoleAlias.AliasBuffer,
                                            Request->Data.GetAllConsoleAlias.AliasBufferLength);
 
     Request->Data.GetAllConsoleAlias.BytesWritten = BytesWritten;
-    Request->Status = STATUS_SUCCESS;
     ConioUnlockConsole(Console);
-    return Request->Status;
+    return STATUS_SUCCESS;
 }
 
 CSR_API(CsrGetAllConsoleAliasesLength)
@@ -495,8 +482,7 @@ CSR_API(CsrGetAllConsoleAliasesLength)
 
     if (Request->Data.GetAllConsoleAliasesLength.lpExeName == NULL)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Request->Status = ConioConsoleFromProcessData(ProcessData, &Console);
@@ -508,17 +494,14 @@ CSR_API(CsrGetAllConsoleAliasesLength)
     Header = IntFindAliasHeader(Console->Aliases, Request->Data.GetAllConsoleAliasesLength.lpExeName);
     if (!Header)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
 
     Length = IntGetAllConsoleAliasesLength(Header);
     Request->Data.GetAllConsoleAliasesLength.Length = Length;
-    Request->Status = STATUS_SUCCESS;
     ConioUnlockConsole(Console);
-    return Request->Status;
-
+    return STATUS_SUCCESS;
 }
 
 CSR_API(CsrGetConsoleAliasesExes)
@@ -526,7 +509,7 @@ CSR_API(CsrGetConsoleAliasesExes)
     PCSRSS_CONSOLE Console;
     UINT BytesWritten;
     UINT ExesLength;
-    
+
     DPRINT("CsrGetConsoleAliasesExes entered\n");
 
     Request->Status = ConioConsoleFromProcessData(ProcessData, &Console);
@@ -536,38 +519,34 @@ CSR_API(CsrGetConsoleAliasesExes)
     }
 
     ExesLength = IntGetConsoleAliasesExesLength(Console->Aliases);
-    
+
     if (ExesLength > Request->Data.GetConsoleAliasesExes.Length)
     {
-        Request->Status = STATUS_BUFFER_OVERFLOW;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_BUFFER_OVERFLOW;
     }
 
     if (Request->Data.GetConsoleAliasesExes.ExeNames == NULL)
     {
-        Request->Status = STATUS_INVALID_PARAMETER;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_INVALID_PARAMETER;
     }
-    
+
     if (!ValidateBuffer(ProcessData,
                         Request->Data.GetConsoleAliasesExes.ExeNames,
                         Request->Data.GetConsoleAliasesExes.Length))
     {
-        Request->Status = STATUS_ACCESS_VIOLATION;
         ConioUnlockConsole(Console);
-        return Request->Status;
+        return STATUS_ACCESS_VIOLATION;
     }
 
-    BytesWritten = IntGetConsoleAliasesExes(Console->Aliases, 
+    BytesWritten = IntGetConsoleAliasesExes(Console->Aliases,
                                             Request->Data.GetConsoleAliasesExes.ExeNames,
                                             Request->Data.GetConsoleAliasesExes.Length);
 
     Request->Data.GetConsoleAliasesExes.BytesWritten = BytesWritten;
-    Request->Status = STATUS_SUCCESS;
     ConioUnlockConsole(Console);
-    return Request->Status;
+    return STATUS_SUCCESS;
 }
 
 CSR_API(CsrGetConsoleAliasesExesLength)