[CONSRV] Fix uninitialized variables warnings detected by Clang. (#3619)
authorHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Wed, 21 Apr 2021 16:45:35 +0000 (18:45 +0200)
committerHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Wed, 5 May 2021 15:24:12 +0000 (17:24 +0200)
CORE-17545

Fix 3 warnings:

win32ss/user/winsrv/consrv/alias.c:648:16: warning: variable 'Status' is used uninitialized whenever 'while' loop exits because its condition is false [-Wsometimes-uninitialized]
        while (CurEntry)
               ^~~~~~~~
win32ss/user/winsrv/consrv/alias.c:693:12: note: uninitialized use occurs here
    return Status;
           ^~~~~~

win32ss/user/winsrv/consrv/alias.c:715:9: warning: variable 'Status' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
    if (Header)
        ^~~~~~
win32ss/user/winsrv/consrv/alias.c:727:12: note: uninitialized use occurs here
    return Status;
           ^~~~~~

win32ss/user/winsrv/consrv/alias.c:771:16: warning: variable 'Status' is used uninitialized whenever 'while' loop exits because its condition is false [-Wsometimes-uninitialized]
        while (RootHeader)
               ^~~~~~~~~~
win32ss/user/winsrv/consrv/alias.c:807:12: note: uninitialized use occurs here
    return Status;

win32ss/user/winsrv/consrv/alias.c

index 8713467..ebe9764 100644 (file)
@@ -594,7 +594,7 @@ CON_API(SrvGetConsoleAlias,
 CON_API(SrvGetConsoleAliases,
         CONSOLE_GETALLALIASES, GetAllAliasesRequest)
 {
-    NTSTATUS Status;
+    NTSTATUS Status = STATUS_SUCCESS;
     ULONG BytesWritten = 0;
     PALIAS_HEADER Header;
 
@@ -697,7 +697,7 @@ Quit:
 CON_API(SrvGetConsoleAliasesLength,
         CONSOLE_GETALLALIASESLENGTH, GetAllAliasesLengthRequest)
 {
-    NTSTATUS Status;
+    NTSTATUS Status = STATUS_SUCCESS;
     PALIAS_HEADER Header;
 
     if (!CsrValidateMessageBuffer(ApiMessage,
@@ -731,7 +731,7 @@ CON_API(SrvGetConsoleAliasesLength,
 CON_API(SrvGetConsoleAliasExes,
         CONSOLE_GETALIASESEXES, GetAliasesExesRequest)
 {
-    NTSTATUS Status;
+    NTSTATUS Status = STATUS_SUCCESS;
     UINT BytesWritten = 0;
 
     if (!CsrValidateMessageBuffer(ApiMessage,