gdiplus: Clean up HRGN when GdipGetRegionBounds succeeds too.

42d1d9e9 · Vincent Povirk · Alexandre Julliard · 296b67f5 · 42d1d9e9
Commit 42d1d9e9 authored Mar 21, 2011 by Vincent Povirk Committed by Alexandre Julliard Mar 28, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 8 deletions

region.c dlls/gdiplus/region.c +9 -8

No files found.
--- a/dlls/gdiplus/region.c
+++ b/dlls/gdiplus/region.c
@@ -683,17 +683,18 @@ GpStatus WINGDIPAPI GdipGetRegionBounds(GpRegion *region, GpGraphics *graphics, 
        return Ok;
    }

-    if(!GetRgnBox(hrgn, &r)){
-        DeleteObject(hrgn);
-        return GenericError;
+    if(GetRgnBox(hrgn, &r)){
+        rect->X = r.left;
+        rect->Y = r.top;
+        rect->Width  = r.right  - r.left;
+        rect->Height = r.bottom - r.top;
    }
+    else
+        status = GenericError;

-    rect->X = r.left;
-    rect->Y = r.top;
-    rect->Width  = r.right  - r.left;
-    rect->Height = r.bottom - r.top;
+    DeleteObject(hrgn);

-    return Ok;
+    return status;
 }

 /*****************************************************************************