[ros-diffs] 02/03: [NTOS/MM] Properly handle page faults in regions marked with PAGE_NOACCESS or PAGE_GUARD

[Jérôme Gardou]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=df25e4e79109a785b4be4…

commit df25e4e79109a785b4be4971081af11573e44259
Author:     Jérôme Gardou <jerome.gardou(a)reactos.org>
AuthorDate: Fri Nov 2 18:23:16 2018 +0100
Commit:     Jérôme Gardou <zefklop(a)users.noreply.github.com>
CommitDate: Sat Nov 3 12:50:16 2018 +0100

    [NTOS/MM] Properly handle page faults in regions marked with
    PAGE_NOACCESS or PAGE_GUARD
    
    ROSTESTS-110
---
 ntoskrnl/mm/section.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/ntoskrnl/mm/section.c b/ntoskrnl/mm/section.c
index 4a03514f00..298fc5f0f9 100644
--- a/ntoskrnl/mm/section.c
+++ b/ntoskrnl/mm/section.c
@@ -1410,6 +1410,29 @@ MmNotPresentFaultSectionView(PMMSUPPORT AddressSpace,
                           &MemoryArea->Data.SectionData.RegionListHead,
                           Address, NULL);
     ASSERT(Region != NULL);
+
+    /* Check for a NOACCESS mapping */
+    if (Region->Protect & PAGE_NOACCESS)
+    {
+        return STATUS_ACCESS_VIOLATION;
+    }
+
+    if (Region->Protect & PAGE_GUARD)
+    {
+        /* Remove it */
+        Status = MmAlterRegion(AddressSpace, (PVOID)MA_GetStartingAddress(MemoryArea),
+                &MemoryArea->Data.SectionData.RegionListHead,
+                Address, PAGE_SIZE, Region->Type, Region->Protect &
~PAGE_GUARD,
+                MmAlterViewAttributes);
+
+        if (!NT_SUCCESS(Status))
+        {
+            DPRINT1("Removing PAGE_GUARD protection failed : 0x%08x.\n",
Status);
+        }
+
+        return STATUS_GUARD_PAGE_VIOLATION;
+    }
+
     /*
      * Lock the segment
      */