[ros-diffs] [tkreuzer] 52535: [NTOSKRNL] - Initialize an uninitialized variable - MI_GET_NEXT_COLOR() doesn't have an argument - Fix a warning on MSVC

[tkreuzer]

Author: tkreuzer
Date: Mon Jul  4 16:26:52 2011
New Revision: 52535

URL: http://svn.reactos.org/svn/reactos?rev=52535&view=rev
Log:
[NTOSKRNL]
- Initialize an uninitialized variable
- MI_GET_NEXT_COLOR() doesn't have an argument
- Fix a warning on MSVC

Modified:
    trunk/reactos/ntoskrnl/mm/ARM3/miarm.h
    trunk/reactos/ntoskrnl/mm/ARM3/miavl.h
    trunk/reactos/ntoskrnl/mm/ARM3/sysldr.c

Modified: trunk/reactos/ntoskrnl/mm/ARM3/miarm.h
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/miarm.h?rev=52535&r1=52534&r2=52535&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] Mon Jul  4 16:26:52 2011
@@ -244,7 +244,7 @@
 // Returns the color of a page
 //
 #define MI_GET_PAGE_COLOR(x)                ((x) & MmSecondaryColorMask)
-#define MI_GET_NEXT_COLOR(x)                
(MI_GET_PAGE_COLOR(++MmSystemPageColor))
+#define MI_GET_NEXT_COLOR()                 
(MI_GET_PAGE_COLOR(++MmSystemPageColor))
 #define MI_GET_NEXT_PROCESS_COLOR(x)        
(MI_GET_PAGE_COLOR(++(x)->NextPageColor))
 
 #ifndef _M_AMD64

Modified: trunk/reactos/ntoskrnl/mm/ARM3/miavl.h
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/miavl.h?rev=52535&r1=52534&r2=52535&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/miavl.h [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/miavl.h [iso-8859-1] Mon Jul  4 16:26:52 2011
@@ -20,7 +20,7 @@
  *
  * The other difference is that the AVL package for Rtl has custom callbacks 
for
  * comparison purposes (which would access some internal, opaque, user data) 
while
- * the Mm package stores the user-data inline as StartingVpn and EndingVpn. So 
+ * the Mm package stores the user-data inline as StartingVpn and EndingVpn. So
  * when a compare is being made, RtlpAvlCompareRoutine is called, which will 
either
  * perform the Mm work, or call the user-specified callback in the Rtl case.
  */
@@ -54,7 +54,7 @@
     {
         return GenericEqual;
     }
-    else 
+    else
     {
         return GenericGreaterThan;
     }
@@ -80,7 +80,7 @@
 FORCEINLINE
 RtlBalance(IN PRTL_BALANCED_LINKS Node)
 {
-    return Node->u1.Balance;
+    return (SCHAR)Node->u1.Balance;
 }
 
 PRTL_BALANCED_LINKS

Modified: trunk/reactos/ntoskrnl/mm/ARM3/sysldr.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/sysldr.c?rev=52535&r1=52534&r2=52535&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/sysldr.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/sysldr.c [iso-8859-1] Mon Jul  4 16:26:52 
2011
@@ -2086,7 +2086,7 @@
     PIMAGE_NT_HEADERS NtHeaders;
     PIMAGE_SECTION_HEADER Section;
     PFN_NUMBER DriverPages;
-    ULONG CurrentProtection, SectionProtection, CombinedProtection, 
ProtectionMask;
+    ULONG CurrentProtection, SectionProtection, CombinedProtection = 0, 
ProtectionMask;
     ULONG Sections, Size;
     ULONG_PTR BaseAddress, CurrentAddress;
     PMMPTE PointerPte, StartPte, LastPte, CurrentPte, ComboPte = NULL;