[ros-diffs] [ion] 60056: [NTOSKRNL]: Fix the fix

Author: ion
Date: Thu Sep 12 07:55:45 2013
New Revision: 60056

URL: http://svn.reactos.org/svn/reactos?rev=60056&view=rev
Log:
[NTOSKRNL]: Fix the fix

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] Thu Sep 12 07:55:45 2013
@@ -4871,17 +4871,6 @@
     ULONG Protection;
     PROS_SECTION_OBJECT *SectionObject = (PROS_SECTION_OBJECT *)Section;

-    /* Convert section flag to page flag */
-    if (AllocationAttributes & SEC_NOCACHE) SectionPageProtection |= PAGE_NOCACHE;
-
-    /* Check to make sure the protection is correct. Nt* does this already */
-    Protection = MiMakeProtectionMask(SectionPageProtection);
-    if (Protection == MM_INVALID_PROTECTION)
-    {
-        DPRINT1("Page protection is invalid\n");
-        return STATUS_INVALID_PAGE_PROTECTION;
-    }
-
     /* Check if an ARM3 section is being created instead */
     if (!(AllocationAttributes & (SEC_IMAGE | SEC_PHYSICALMEMORY)))
     {
@@ -4896,6 +4885,17 @@
                                        FileHandle,
                                        FileObject);
         }
+    }
+
+    /* Convert section flag to page flag */
+    if (AllocationAttributes & SEC_NOCACHE) SectionPageProtection |= PAGE_NOCACHE;
+
+    /* Check to make sure the protection is correct. Nt* does this already */
+    Protection = MiMakeProtectionMask(SectionPageProtection);
+    if (Protection == MM_INVALID_PROTECTION)
+    {
+        DPRINT1("Page protection is invalid\n");
+        return STATUS_INVALID_PAGE_PROTECTION;
     }

/* Check if this is going to be a data or image backed file section */