user32: Free the correct dce (Coverity).

3fb8a426 · Marcus Meissner · Alexandre Julliard · f1b4c331 · 3fb8a426
Commit 3fb8a426 authored Aug 17, 2012 by Marcus Meissner Committed by Alexandre Julliard Aug 20, 2012
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

painting.c dlls/user32/painting.c +5 -3

No files found.
--- a/dlls/user32/painting.c
+++ b/dlls/user32/painting.c
@@ -314,9 +314,11 @@ static struct dce *get_window_dce( HWND hwnd )

        if (dce_to_free)
        {
-            SetDCHook( dce->hdc, NULL, 0 );
-            DeleteDC( dce->hdc );
-            HeapFree( GetProcessHeap(), 0, dce );
+            SetDCHook( dce_to_free->hdc, NULL, 0 );
+            DeleteDC( dce_to_free->hdc );
+            HeapFree( GetProcessHeap(), 0, dce_to_free );
+            if (dce_to_free == dce)
+                dce = NULL;
        }
    }
    return dce;