fixed some warnings when compiling with -O3

svn path=/trunk/; revision=16928

diff --git a/reactos/drivers/bus/serenum/detect.c b/reactos/drivers/bus/serenum/detect.c
index ca64d2c..a2813c2 100644 (file)
--- a/reactos/drivers/bus/serenum/detect.c
+++ b/reactos/drivers/bus/serenum/detect.c
@@ -472,7 +472,7 @@ SerenumDetectLegacyDevice(
        ULONG Command;
        SERIAL_TIMEOUTS Timeouts;
        SERIAL_LINE_CONTROL LCR;
-       ULONG i, Count;
+       ULONG i, Count = 0;
        UCHAR Buffer[16];
        UNICODE_STRING DeviceDescription;
        UNICODE_STRING DeviceId;

diff --git a/reactos/drivers/dd/sndblst/irq.c b/reactos/drivers/dd/sndblst/irq.c
index b9ffbff..efa09ff 100644 (file)
--- a/reactos/drivers/dd/sndblst/irq.c
+++ b/reactos/drivers/dd/sndblst/irq.c
@@ -65,7 +65,7 @@ NTSTATUS EnableIRQ(PDEVICE_OBJECT DeviceObject)
     ULONG Vector;
     KIRQL IRQ_Level;
     KAFFINITY Affinity;
-    NTSTATUS Status;
+    NTSTATUS Status = STATUS_SUCCESS;
 
     Vector = HalGetInterruptVector(Isa,     // FIX THIS
                                    0,       // FIX THIS

diff --git a/reactos/drivers/input/i8042prt/i8042prt.c b/reactos/drivers/input/i8042prt/i8042prt.c
index b3593c3..ce3324d 100644 (file)
--- a/reactos/drivers/input/i8042prt/i8042prt.c
+++ b/reactos/drivers/input/i8042prt/i8042prt.c
@@ -511,7 +511,7 @@ NTSTATUS STDCALL I8042CreateDispatch(PDEVICE_OBJECT DeviceObject, PIRP Irp)
 static NTSTATUS STDCALL I8042BasicDetect(PDEVICE_EXTENSION DevExt)
 {
        NTSTATUS Status;
-       UCHAR Value;
+       UCHAR Value = 0;
        UINT Counter;
 
        I8042Flush();

diff --git a/reactos/drivers/input/i8042prt/mouse.c b/reactos/drivers/input/i8042prt/mouse.c
index 50c429a..a4664e5 100644 (file)
--- a/reactos/drivers/input/i8042prt/mouse.c
+++ b/reactos/drivers/input/i8042prt/mouse.c
@@ -111,7 +111,7 @@ BOOLEAN STDCALL I8042MouseResetIsr(PDEVICE_EXTENSION DevExt,
                                    UCHAR Status,
                                    PUCHAR Value)
 {
-       BOOLEAN ToReturn;
+       BOOLEAN ToReturn = FALSE;
 
        if (I8042MouseCallIsrHook(DevExt, Status, Value, &ToReturn))
                return ToReturn;

diff --git a/reactos/lib/advapi32/token/token.c b/reactos/lib/advapi32/token/token.c
index c58657d..bd4a95b 100644 (file)
--- a/reactos/lib/advapi32/token/token.c
+++ b/reactos/lib/advapi32/token/token.c
@@ -327,8 +327,7 @@ CheckTokenMembership (HANDLE ExistingTokenHandle,
                       PSID SidToCheck,
                       PBOOL IsMember)
 {
-  HANDLE AccessToken;
-  BOOL ReleaseToken = FALSE;
+  HANDLE AccessToken = NULL;
   BOOL Result = FALSE;
   DWORD dwSize;
   DWORD i;
@@ -353,7 +352,6 @@ CheckTokenMembership (HANDLE ExistingTokenHandle,
       goto ByeBye;
     }
     CloseHandle(ExistingTokenHandle);
-    ReleaseToken = TRUE;
   }
   else
   {
@@ -364,7 +362,6 @@ CheckTokenMembership (HANDLE ExistingTokenHandle,
       /* Duplicate token to have a impersonation token */
       if (!DuplicateToken(ExistingTokenHandle, SecurityAnonymous, &AccessToken))
         return FALSE;
-      ReleaseToken = TRUE;
     }
     else
       AccessToken = ExistingTokenHandle;
@@ -395,7 +392,7 @@ CheckTokenMembership (HANDLE ExistingTokenHandle,
 ByeBye:
   if (lpGroups != NULL)
     HeapFree(GetProcessHeap(), 0, lpGroups);
-  if (ReleaseToken)
+  if (AccessToken != NULL && AccessToken != ExistingTokenHandle)
     CloseHandle(AccessToken);
 
   return Result;

diff --git a/reactos/lib/kernel32/misc/time.c b/reactos/lib/kernel32/misc/time.c
index 7a7b3f3..3b90f2e 100644 (file)
--- a/reactos/lib/kernel32/misc/time.c
+++ b/reactos/lib/kernel32/misc/time.c
@@ -50,7 +50,7 @@ FileTimeToDosDateTime(
 {
    PDOSTIME  pdtime=(PDOSTIME) lpFatTime;
    PDOSDATE  pddate=(PDOSDATE) lpFatDate;
-   SYSTEMTIME SystemTime;
+   SYSTEMTIME SystemTime = { 0 };
 
    if ( lpFileTime == NULL )
                return FALSE;

diff --git a/reactos/ntoskrnl/cm/regfile.c b/reactos/ntoskrnl/cm/regfile.c
index 9ec1dd9..7cb510c 100644 (file)
--- a/reactos/ntoskrnl/cm/regfile.c
+++ b/reactos/ntoskrnl/cm/regfile.c
@@ -4065,6 +4065,10 @@ CmiCopyKey (PREGISTRY_HIVE DstHive,
        }
       NewKeyCell->HashTableOffset = NewHashTableOffset;
     }
+  else
+    {
+      NewHashTableCell = NULL;
+    }
 
   /* Allocate and copy value list and values */
   if (SrcKeyCell->NumberOfValues != 0)

diff --git a/reactos/ntoskrnl/mm/elf.inc.h b/reactos/ntoskrnl/mm/elf.inc.h
index a15f06a..3d3e27e 100644 (file)
--- a/reactos/ntoskrnl/mm/elf.inc.h
+++ b/reactos/ntoskrnl/mm/elf.inc.h
@@ -519,8 +519,8 @@ Elf64FmtCreateSection
     Elf_Off nFileOffset;
     Elf_Addr nVirtualAddr;
     Elf_Size nAdj;
-    Elf_Size nVirtualSize;
-    Elf_Size nFileSize;
+    Elf_Size nVirtualSize = 0;
+    Elf_Size nFileSize = 0;
 
     ASSERT(j <= nPHdrCount);
 

diff --git a/reactos/ntoskrnl/mm/pe.c b/reactos/ntoskrnl/mm/pe.c
index bb4d19d..b863d18 100644 (file)
--- a/reactos/ntoskrnl/mm/pe.c
+++ b/reactos/ntoskrnl/mm/pe.c
@@ -188,12 +188,12 @@ NTSTATUS NTAPI PeFmtCreateSection
 )
 {
  NTSTATUS nStatus;
- ULONG cbFileHeaderOffsetSize;
- ULONG cbSectionHeadersOffset;
+ ULONG cbFileHeaderOffsetSize = 0;
+ ULONG cbSectionHeadersOffset = 0;
  ULONG cbSectionHeadersSize;
- ULONG cbSectionHeadersOffsetSize;
+ ULONG cbSectionHeadersOffsetSize = 0;
  ULONG cbOptHeaderSize;
- ULONG cbHeadersSize;
+ ULONG cbHeadersSize = 0;
  ULONG nSectionAlignment;
  ULONG nFileAlignment;
  const IMAGE_DOS_HEADER * pidhDosHeader;
@@ -203,8 +203,8 @@ NTSTATUS NTAPI PeFmtCreateSection
  PMM_SECTION_SEGMENT pssSegments;
  LARGE_INTEGER lnOffset;
  PVOID pBuffer;
- ULONG nPrevVirtualEndOfSegment;
- ULONG nFileSizeOfHeaders;
+ ULONG nPrevVirtualEndOfSegment = 0;
+ ULONG nFileSizeOfHeaders = 0;
  ULONG i;
 
  ASSERT(FileHeader);
@@ -277,6 +277,7 @@ NTSTATUS NTAPI PeFmtCreateSection
   PVOID pData;
 
 l_ReadHeaderFromFile:
+  cbNtHeaderSize = 0;
   lnOffset.QuadPart = pidhDosHeader->e_lfanew;
 
   /* read the header from the file */
@@ -330,7 +331,7 @@ l_ReadHeaderFromFile:
  }
  else
  {
-  ULONG cbOptHeaderOffsetSize;
+  ULONG cbOptHeaderOffsetSize = 0;
 
   nStatus = STATUS_INVALID_IMAGE_FORMAT;
 

diff --git a/reactos/ntoskrnl/mm/region.c b/reactos/ntoskrnl/mm/region.c
index 5ee161a..03427db 100644 (file)
--- a/reactos/ntoskrnl/mm/region.c
+++ b/reactos/ntoskrnl/mm/region.c
@@ -110,7 +110,7 @@ MmAlterRegion(PMADDRESS_SPACE AddressSpace, PVOID BaseAddress,
               ULONG NewType, ULONG NewProtect, PMM_ALTER_REGION_FUNC AlterFunc)
 {
    PMM_REGION InitialRegion;
-   PVOID InitialBaseAddress;
+   PVOID InitialBaseAddress = NULL;
    PMM_REGION NewRegion;
    PLIST_ENTRY CurrentEntry;
    PMM_REGION CurrentRegion = NULL;

diff --git a/reactos/subsys/csrss/win32csr/conio.c b/reactos/subsys/csrss/win32csr/conio.c
index 2ec7b45..62b8a89 100644 (file)
--- a/reactos/subsys/csrss/win32csr/conio.c
+++ b/reactos/subsys/csrss/win32csr/conio.c
@@ -224,7 +224,7 @@ CsrInitConsole(PCSRSS_CONSOLE Console)
 CSR_API(CsrAllocConsole)
 {
     PCSRSS_CONSOLE Console;
-    NTSTATUS Status;
+    NTSTATUS Status = STATUS_SUCCESS;
     BOOLEAN NewConsole = FALSE;
 
     DPRINT("CsrAllocConsole\n");

diff --git a/reactos/subsys/system/regedit/regproc.c b/reactos/subsys/system/regedit/regproc.c
index cf0c133..9d597f8 100644 (file)
--- a/reactos/subsys/system/regedit/regproc.c
+++ b/reactos/subsys/system/regedit/regproc.c
@@ -384,7 +384,7 @@ void REGPROC_unescape_string(LPSTR str)
 HRESULT setValue(LPSTR val_name, LPSTR val_data)
 {
     HRESULT hRes;
-    DWORD   dwDataType, dwParseType;
+    DWORD   dwDataType, dwParseType = REG_BINARY;
     LPBYTE lpbData;
     BYTE   convert[KEY_MAX_LEN];
     BYTE *bBigBuffer = 0;