[NTOSKRNL]: Warn, but do not ASSERT, about the fact that we don't have commit charges yet. Fixes OllyDbg2 assertions (and probably others too).

svn path=/trunk/; revision=57223
This commit is contained in:
Alex Ionescu 2012-09-02 20:56:55 +00:00
parent 74f2cde659
commit c37fa08214

View file

@ -94,8 +94,8 @@ C_ASSERT(SYSTEM_PD_SIZE == PAGE_SIZE);
#endif #endif
// //
// Protection Bits part of the internal memory manager Protection Mask // Protection Bits part of the internal memory manager Protection Mask, from:
// Taken from http://www.reactos.org/wiki/Techwiki:Memory_management_in_the_Windows_XP_kernel // http://reactos.org/wiki/Techwiki:Memory_management_in_the_Windows_XP_kernel
// and public assertions. // and public assertions.
// //
#define MM_ZERO_ACCESS 0 #define MM_ZERO_ACCESS 0
@ -1263,8 +1263,8 @@ MiDropLockCount(IN PMMPFN Pfn1)
if ((Pfn1->u3.e1.PrototypePte == 1) && if ((Pfn1->u3.e1.PrototypePte == 1) &&
(Pfn1->OriginalPte.u.Soft.Prototype == 1)) (Pfn1->OriginalPte.u.Soft.Prototype == 1))
{ {
/* We don't handle this */ /* FIXME: We should return commit */
ASSERT(FALSE); DPRINT1("Not returning commit for prototype PTE\n");
} }
/* Update the counter */ /* Update the counter */
@ -1301,8 +1301,8 @@ MiDereferencePfnAndDropLockCount(IN PMMPFN Pfn1)
if ((Pfn1->u3.e1.PrototypePte == 1) && if ((Pfn1->u3.e1.PrototypePte == 1) &&
(Pfn1->OriginalPte.u.Soft.Prototype == 1)) (Pfn1->OriginalPte.u.Soft.Prototype == 1))
{ {
/* We don't handle this */ /* FIXME: We should return commit */
ASSERT(FALSE); DPRINT1("Not returning commit for prototype PTE\n");
} }
/* Update the counter, and drop a reference the long way */ /* Update the counter, and drop a reference the long way */
@ -1407,8 +1407,8 @@ MiReferenceUsedPageAndBumpLockCount(IN PMMPFN Pfn1)
if ((Pfn1->u3.e1.PrototypePte == 1) && if ((Pfn1->u3.e1.PrototypePte == 1) &&
(Pfn1->OriginalPte.u.Soft.Prototype == 1)) (Pfn1->OriginalPte.u.Soft.Prototype == 1))
{ {
/* We don't handle this */ /* FIXME: We should charge commit */
ASSERT(FALSE); DPRINT1("Not charging commit for prototype PTE\n");
} }
/* More locked pages! */ /* More locked pages! */
@ -1456,8 +1456,8 @@ MiReferenceUnusedPageAndBumpLockCount(IN PMMPFN Pfn1)
if ((Pfn1->u3.e1.PrototypePte == 1) && if ((Pfn1->u3.e1.PrototypePte == 1) &&
(Pfn1->OriginalPte.u.Soft.Prototype == 1)) (Pfn1->OriginalPte.u.Soft.Prototype == 1))
{ {
/* We don't handle this */ /* FIXME: We should charge commit */
ASSERT(FALSE); DPRINT1("Not charging commit for prototype PTE\n");
} }
/* More locked pages! */ /* More locked pages! */