ieframe/tests: Don't crash if IWebBrowser2_get_LocationURL() fails.

19e2c157 · Francois Gouget · Alexandre Julliard · 9fe25eae · 19e2c157
Commit 19e2c157 authored Sep 16, 2011 by Francois Gouget Committed by Alexandre Julliard Sep 16, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

webbrowser.c dlls/ieframe/tests/webbrowser.c +5 -2

No files found.
--- a/dlls/ieframe/tests/webbrowser.c
+++ b/dlls/ieframe/tests/webbrowser.c
@@ -230,9 +230,12 @@ static void _test_LocationURL(unsigned line, IUnknown *unk, const char *exurl)

    hres = IWebBrowser2_get_LocationURL(wb, &url);
    ok_(__FILE__,line) (hres == (*exurl ? S_OK : S_FALSE), "get_LocationURL failed: %08x\n", hres);
-    ok_(__FILE__,line) (!strcmp_wa(url, exurl), "unexpected URL: %s\n", wine_dbgstr_w(url));
+    if (hres == S_OK)
+    {
+        ok_(__FILE__,line) (!strcmp_wa(url, exurl), "unexpected URL: %s\n", wine_dbgstr_w(url));
+        SysFreeString(url);
+    }

-    SysFreeString(url);
    IWebBrowser2_Release(wb);
 }