[ros-diffs] [tkreuzer] 61073: [NTOSKRNL] Fix an off-by-one error in an ASSERT in MiDecrementReferenceCount (MmHighestPhysicalPage itself is still a valid PFN)

Author: tkreuzer
Date: Fri Nov 22 12:10:20 2013
New Revision: 61073

URL: http://svn.reactos.org/svn/reactos?rev=61073&view=rev
Log:
[NTOSKRNL]
Fix an off-by-one error in an ASSERT in MiDecrementReferenceCount (MmHighestPhysicalPage itself is still a valid PFN)

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

Modified: trunk/reactos/ntoskrnl/mm/ARM3/pfnlist.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/pfnlist.c?...
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/pfnlist.c	[iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/pfnlist.c	[iso-8859-1] Fri Nov 22 12:10:20 2013
@@ -1308,14 +1308,14 @@
     ASSERT(KeGetCurrentIrql() == DISPATCH_LEVEL);

/* Sanity checks on the page */
-    if (PageFrameIndex >= MmHighestPhysicalPage ||
+    if (PageFrameIndex > MmHighestPhysicalPage ||
         Pfn1 != MI_PFN_ELEMENT(PageFrameIndex) ||
         Pfn1->u3.e2.ReferenceCount == 0 ||
         Pfn1->u3.e2.ReferenceCount >= 2500)
     {
         DPRINT1("PageFrameIndex=0x%lx, MmHighestPhysicalPage=0x%lx\n", PageFrameIndex, MmHighestPhysicalPage);
         DPRINT1("Pfn1=%p, Element=%p, RefCount=%u\n", Pfn1, MI_PFN_ELEMENT(PageFrameIndex), Pfn1->u3.e2.ReferenceCount);
-        ASSERT(PageFrameIndex < MmHighestPhysicalPage);
+        ASSERT(PageFrameIndex <= MmHighestPhysicalPage);
         ASSERT(Pfn1 == MI_PFN_ELEMENT(PageFrameIndex));
         ASSERT(Pfn1->u3.e2.ReferenceCount != 0);
         ASSERT(Pfn1->u3.e2.ReferenceCount < 2500);