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

Author: ion
Date: Sun Sep  2 20:56:55 2012
New Revision: 57223

URL: http://svn.reactos.org/svn/reactos?rev=57223&view=rev
Log:
[NTOSKRNL]: Warn, but do not ASSERT, about the fact that we don't have commit charges yet. Fixes OllyDbg2 assertions (and probably others too).

Modified:
    trunk/reactos/ntoskrnl/mm/ARM3/miarm.h

Modified: trunk/reactos/ntoskrnl/mm/ARM3/miarm.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/miarm.h?re...
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] Sun Sep  2 20:56:55 2012
@@ -94,8 +94,8 @@
 #endif

//
-// Protection Bits part of the internal memory manager Protection Mask
-// Taken from http://www.reactos.org/wiki/Techwiki:Memory_management_in_the_Windows_XP_ker...
+// Protection Bits part of the internal memory manager Protection Mask, from:
+// http://reactos.org/wiki/Techwiki:Memory_management_in_the_Windows_XP_kernel
 // and public assertions.
 //
 #define MM_ZERO_ACCESS         0
@@ -1263,8 +1263,8 @@
         if ((Pfn1->u3.e1.PrototypePte == 1) &&
             (Pfn1->OriginalPte.u.Soft.Prototype == 1))
         {
-            /* We don't handle this */
-            ASSERT(FALSE);
+            /* FIXME: We should return commit */
+            DPRINT1("Not returning commit for prototype PTE\n");
         }

/* Update the counter */
@@ -1301,8 +1301,8 @@
             if ((Pfn1->u3.e1.PrototypePte == 1) &&
                 (Pfn1->OriginalPte.u.Soft.Prototype == 1))
             {
-                /* We don't handle this */
-                ASSERT(FALSE);
+                /* FIXME: We should return commit */
+                DPRINT1("Not returning commit for prototype PTE\n");
             }

/* Update the counter, and drop a reference the long way */
@@ -1407,8 +1407,8 @@
     if ((Pfn1->u3.e1.PrototypePte == 1) &&
         (Pfn1->OriginalPte.u.Soft.Prototype == 1))
     {
-        /* We don't handle this */
-        ASSERT(FALSE);
+        /* FIXME: We should charge commit */
+        DPRINT1("Not charging commit for prototype PTE\n");
     }

/* More locked pages! */
@@ -1456,8 +1456,8 @@
     if ((Pfn1->u3.e1.PrototypePte == 1) &&
         (Pfn1->OriginalPte.u.Soft.Prototype == 1))
     {
-        /* We don't handle this */
-        ASSERT(FALSE);
+        /* FIXME: We should charge commit */
+        DPRINT1("Not charging commit for prototype PTE\n");
     }

/* More locked pages! */