+sanity-check of the value of 'KeGetCurrentThread()->KernelApcDisable'

KeEnterCriticalRegion() vs. KeLeaveCriticalRegion() now ++/-- by reverse
 - *Enter*() now adds, *Leave*() now subs; why the other way?

diff --git a/ntoskrnl/ke/critical.c b/ntoskrnl/ke/critical.c
index d6179ab..6ee8326 100644 (file)
--- a/ntoskrnl/ke/critical.c
+++ b/ntoskrnl/ke/critical.c
@@ -22,13 +22,17 @@
 VOID STDCALL KeEnterCriticalRegion (VOID)
 {
    DPRINT("KeEnterCriticalRegion()\n");
-   KeGetCurrentThread()->KernelApcDisable -= 1;
+   if (KeGetCurrentThread()->KernelApcDisable<0)
+      KeBugCheck(0);
+   KeGetCurrentThread()->KernelApcDisable += 1;
 }
 
 VOID STDCALL KeLeaveCriticalRegion (VOID)
 {
    DPRINT("KeLeaveCriticalRegion()\n");
-   KeGetCurrentThread()->KernelApcDisable += 1;
+   if (KeGetCurrentThread()->KernelApcDisable<=0)
+      KeBugCheck(0);
+   KeGetCurrentThread()->KernelApcDisable -= 1;
 }
 
 /* EOF */