Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-08-05 04:23:32 +00:00
Code Issues Projects Releases Packages Wiki Activity

[NTOS:MM]

Browse source 
- Generate PAGE_FAULT_IN_NONPAGED_AREA bugcheck for PTEs that have their protection set to zero. This gives a more useful error description than the infamous 'TempPte.u.Long != 0' assertion*.
CORE-8679 #resolve

* and is also What Windows Does(TM)

svn path=/trunk/; revision=64885
This commit is contained in:
Thomas Faber 2014-10-22 12:29:31 +00:00
parent a48fe43631
commit f67ab25a16
1 changed files with 22 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

33
reactos/ntoskrnl/mm/ARM3/pagfault.c
View file

@ -935,15 +935,15 @@ MiResolveTransitionFault(IN PVOID FaultingAddress,
/* See if we should wait before terminating the fault */ /* See if we should wait before terminating the fault */
if (Pfn1->u3.e1.ReadInProgress == 1) if (Pfn1->u3.e1.ReadInProgress == 1)
{ {
DPRINT1("The page is currently being read!\n"); DPRINT1("The page is currently being read!\n");
ASSERT(Pfn1->u1.Event != NULL); ASSERT(Pfn1->u1.Event != NULL);
*InPageBlock = Pfn1->u1.Event; *InPageBlock = Pfn1->u1.Event;
if (PointerPte == Pfn1->PteAddress) if (PointerPte == Pfn1->PteAddress)
{ {
DPRINT1("And this if for this particular PTE.\n"); DPRINT1("And this if for this particular PTE.\n");
/* The PTE will be made valid by the thread serving the fault */ /* The PTE will be made valid by the thread serving the fault */
return STATUS_SUCCESS; // FIXME: Maybe something more descriptive return STATUS_SUCCESS; // FIXME: Maybe something more descriptive
} }
} }
/* Windows checks there's some free pages and this isn't an in-page error */ /* Windows checks there's some free pages and this isn't an in-page error */
@ -1424,8 +1424,8 @@ MiDispatchFault(IN BOOLEAN StoreInstruction,
if (InPageBlock != NULL) if (InPageBlock != NULL)
{ {
/* The page is being paged in by another process */ /* The page is being paged in by another process */
KeWaitForSingleObject(InPageBlock, WrPageIn, KernelMode, FALSE, NULL); KeWaitForSingleObject(InPageBlock, WrPageIn, KernelMode, FALSE, NULL);
} }
ASSERT(OldIrql == KeGetCurrentIrql()); ASSERT(OldIrql == KeGetCurrentIrql());
@ -1833,6 +1833,17 @@ _WARN("Session space stuff is not implemented yet!")
(ULONG_PTR)TrapInformation, (ULONG_PTR)TrapInformation,
1); 1);
} }
/* Check for no protecton at all */
if (TempPte.u.Soft.Protection == MM_ZERO_ACCESS)
{
/* Bugcheck the system! */
KeBugCheckEx(PAGE_FAULT_IN_NONPAGED_AREA,
(ULONG_PTR)Address,
StoreInstruction,
(ULONG_PTR)TrapInformation,
0);
}
} }
/* Check for demand page */ /* Check for demand page */