winedbg: Don't access inside a bp structure if it's not initialized.

Spotted by Peter Urbanec.

winedbg: Don't access inside a bp structure if it's not initialized.
aebb9dd3 · Eric Pouech · Alexandre Julliard · 1e0af22a · aebb9dd3
Commit aebb9dd3 authored Aug 25, 2010 by Eric Pouech Committed by Alexandre Julliard Aug 26, 2010
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 4 deletions

break.c programs/winedbg/break.c +6 -4

No files found.
--- a/programs/winedbg/break.c
+++ b/programs/winedbg/break.c
@@ -510,11 +510,13 @@ void break_delete_xpoints_from_module(DWORD64 base)

    for (i = 0; i < dbg_curr_process->next_bp; i++)
    {
-        linear = (DWORD_PTR)memory_to_linear_addr(&bp[i].addr);
-        if (bp[i].refcount && bp[i].enabled &&
-            im.BaseOfImage <= linear && linear < im.BaseOfImage + im.ImageSize)
+        if (bp[i].refcount && bp[i].enabled)
        {
-            break_delete_xpoint(i);
+            linear = (DWORD_PTR)memory_to_linear_addr(&bp[i].addr);
+            if (im.BaseOfImage <= linear && linear < im.BaseOfImage + im.ImageSize)
+            {
+                break_delete_xpoint(i);
+            }
        }
    }
 }