gdi32/tests: Test the return value of GetClipBox before trying to access the rect.

16c20d3c · Huw Davies · Alexandre Julliard · 625ec9f0 · 16c20d3c
Commit 16c20d3c authored Nov 13, 2009 by Huw Davies Committed by Alexandre Julliard Nov 13, 2009
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

metafile.c dlls/gdi32/tests/metafile.c +5 -2

No files found.
--- a/dlls/gdi32/tests/metafile.c
+++ b/dlls/gdi32/tests/metafile.c
@@ -1953,8 +1953,11 @@ static void test_emf_clipping(void)
    SetRect(&rc_sclip, 100, 100, GetSystemMetrics(SM_CXSCREEN), GetSystemMetrics(SM_CYSCREEN));
    hrgn = CreateRectRgn(rc_sclip.left, rc_sclip.top, rc_sclip.right, rc_sclip.bottom);
    SelectClipRgn(hdc, hrgn);
-    GetClipBox(hdc, &rc_res);
+    ret = GetClipBox(hdc, &rc_res);
-    todo_wine ok(EqualRect(&rc_res, &rc_sclip),
+todo_wine
+    ok(ret == SIMPLEREGION, "got %d\n", ret);
+    if(ret == SIMPLEREGION)
+        ok(EqualRect(&rc_res, &rc_sclip),
                 "expected rc_res (%d, %d) - (%d, %d), got (%d, %d) - (%d, %d)\n",
                 rc_sclip.left, rc_sclip.top, rc_sclip.right, rc_sclip.bottom,
                 rc_res.left, rc_res.top, rc_res.right, rc_res.bottom);