[ros-diffs] [tkreuzer] 53525: [NTOSKRNL] Fix condition check for informing the debugger in NtMapViewOfSection Found by Amine Khaldi

Author: tkreuzer
Date: Thu Sep  1 18:55:12 2011
New Revision: 53525

URL: http://svn.reactos.org/svn/reactos?rev=53525&view=rev
Log:
[NTOSKRNL]
Fix condition check for informing the debugger in NtMapViewOfSection
Found by Amine Khaldi

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

Modified: trunk/reactos/ntoskrnl/mm/ARM3/section.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/section.c?...
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] Thu Sep  1 18:55:12 2011
@@ -1679,22 +1679,21 @@
                                 AllocationType,
                                 Protect);

-    /* Check if this is an image for the current process */
-    if ((Section->AllocationAttributes & SEC_IMAGE) &&
-        (Process == PsGetCurrentProcess()) &&
-        ((Status != STATUS_IMAGE_NOT_AT_BASE) ||
-         (Status != STATUS_CONFLICTING_ADDRESSES)))
-    {
-        /* Notify the debugger */
-        DbgkMapViewOfSection(Section,
-                             SafeBaseAddress,
-                             SafeSectionOffset.LowPart,
-                             SafeViewSize);
-    }
-
     /* Return data only on success */
     if (NT_SUCCESS(Status))
     {
+        /* Check if this is an image for the current process */
+        if ((Section->AllocationAttributes & SEC_IMAGE) &&
+            (Process == PsGetCurrentProcess()) &&
+            (Status != STATUS_IMAGE_NOT_AT_BASE))
+        {
+            /* Notify the debugger */
+            DbgkMapViewOfSection(Section,
+                                 SafeBaseAddress,
+                                 SafeSectionOffset.LowPart,
+                                 SafeViewSize);
+        }
+
         /* Enter SEH */
         _SEH2_TRY
         {