[PATCH v2 3/6] mshtml: Handle S_FALSE from IUri methods when checking targetOrigin.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/mshtml/htmlwindow.c | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/dlls/mshtml/htmlwindow.c b/dlls/mshtml/htmlwindow.c
index 4cc9852e48f..96ac743b8b6 100644
--- a/dlls/mshtml/htmlwindow.c
+++ b/dlls/mshtml/htmlwindow.c
@@ -3179,24 +3179,32 @@ static HRESULT check_target_origin(HTMLInnerWindow *window, const WCHAR *target_
         goto done;
     }

+    bstr = NULL;
     hres = IUri_GetSchemeName(uri, &bstr);
-    if(FAILED(hres))
+    if(hres != S_OK) {
+        SysFreeString(bstr);
         goto done;
+    }
     hres = IUri_GetSchemeName(target, &bstr2);
     if(SUCCEEDED(hres)) {
-        hres = !wcsicmp(bstr, bstr2) ? S_OK : S_FALSE;
+        if(hres == S_OK && wcsicmp(bstr, bstr2))
+            hres = S_FALSE;
         SysFreeString(bstr2);
     }
     SysFreeString(bstr);
     if(hres != S_OK)
         goto done;

+    bstr = NULL;
     hres = IUri_GetHost(uri, &bstr);
-    if(FAILED(hres))
+    if(hres != S_OK) {
+        SysFreeString(bstr);
         goto done;
+    }
     hres = IUri_GetHost(target, &bstr2);
     if(SUCCEEDED(hres)) {
-        hres = !wcsicmp(bstr, bstr2) ? S_OK : S_FALSE;
+        if(hres == S_OK && wcsicmp(bstr, bstr2))
+            hres = S_FALSE;
         SysFreeString(bstr2);
     }
     SysFreeString(bstr);
@@ -3208,10 +3216,10 @@ static HRESULT check_target_origin(HTMLInnerWindow *window, const WCHAR *target_
         goto done;

hres = IUri_GetPort(uri, &port);
-    if(FAILED(hres))
+    if(hres != S_OK)
         goto done;
     hres = IUri_GetPort(target, &port2);
-    if(SUCCEEDED(hres))
+    if(hres == S_OK)
         hres = (port == port2) ? S_OK : S_FALSE;

done: