- Fix a "clever" check in KdpDeleteBreakpointRange that made the routine unable to...

[reactos.git] / reactos / ntoskrnl / kd64 / kdbreak.c

diff --git a/reactos/ntoskrnl/kd64/kdbreak.c b/reactos/ntoskrnl/kd64/kdbreak.c
index 6b3aa54..a75fe57 100644 (file)
--- a/reactos/ntoskrnl/kd64/kdbreak.c
+++ b/reactos/ntoskrnl/kd64/kdbreak.c
@@ -206,7 +206,10 @@ KdpDeleteBreakpointRange(IN PVOID Base,
                          IN PVOID Limit)
 {
     ULONG BpIndex;
-    BOOLEAN Return = FALSE;
+    BOOLEAN DeletedBreakpoints;
+
+    /* Assume no breakpoints will be deleted */
+    DeletedBreakpoints = FALSE;
 
     /* Loop the breakpoint table */
     for (BpIndex = 0; BpIndex < KD_BREAKPOINT_MAX; BpIndex++)
@@ -216,13 +219,13 @@ KdpDeleteBreakpointRange(IN PVOID Base,
             ((KdpBreakpointTable[BpIndex].Address >= Base) &&
              (KdpBreakpointTable[BpIndex].Address <= Limit)))
         {
-            /* Delete it */
-            Return = Return || KdpDeleteBreakpoint(BpIndex + 1);
+            /* Delete it, and remember if we succeeded at least once */
+            if (KdpDeleteBreakpoint(BpIndex + 1)) DeletedBreakpoints = TRUE;
         }
     }
 
-    /* Return to caller */
-    return Return;
+    /* Return whether we deleted anything */
+    return DeletedBreakpoints;
 }
 
 VOID