[tfaber] 68593: [NTOS:MM] - Don't accept 64 bit PE files on x86 (will cause use of uninitialized variable ImageBase). CORE-9955 - Remove a pointless check in MmMapViewOfSection

Author: tfaber
Date: Sun Aug  2 09:15:39 2015
New Revision: 68593

URL: http://svn.reactos.org/svn/reactos?rev=68593&view=rev
Log:
[NTOS:MM]
- Don't accept 64 bit PE files on x86 (will cause use of uninitialized variable ImageBase). CORE-9955
- Remove a pointless check in MmMapViewOfSection

Modified:
    trunk/reactos/ntoskrnl/mm/section.c

Modified: trunk/reactos/ntoskrnl/mm/section.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=6...
==============================================================================
--- trunk/reactos/ntoskrnl/mm/section.c	[iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/section.c	[iso-8859-1] Sun Aug  2 09:15:39 2015
@@ -363,7 +363,9 @@
     switch(piohOptHeader->Magic)
     {
     case IMAGE_NT_OPTIONAL_HDR32_MAGIC:
+#ifdef _WIN64
     case IMAGE_NT_OPTIONAL_HDR64_MAGIC:
+#endif // _WIN64
         break;

default:
@@ -4609,13 +4611,6 @@
         {
             MmUnlockAddressSpace(AddressSpace);
             return STATUS_SECTION_PROTECTION;
-        }
-
-        if (ViewSize == NULL)
-        {
-            /* Following this pointer would lead to us to the dark side */
-            /* What to do? Bugcheck? Return status? Do the mambo? */
-            KeBugCheck(MEMORY_MANAGEMENT);
         }

if (SectionOffset == NULL)