Alex Henrie : mshtml: Fix use after free in use_gecko_script (Coverity).

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 4ef2557d87e363a1ed38083f9b1ed413bff76d26
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=4ef2557d87e363a1ed38083f9...

Author: Alex Henrie 
Date:   Wed Dec 29 22:56:38 2021 -0700

mshtml: Fix use after free in use_gecko_script (Coverity).

Signed-off-by: Alex Henrie 
Signed-off-by: Jacek Caban 
Signed-off-by: Alexandre Julliard

---

dlls/mshtml/script.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c
index 28e07a50c8e..2a0d86c9f76 100644
--- a/dlls/mshtml/script.c
+++ b/dlls/mshtml/script.c
@@ -1716,12 +1716,13 @@ static BOOL use_gecko_script(IUri *uri)
         return FALSE;

hres = IInternetSecurityManager_MapUrlToZone(get_security_manager(), display_uri, &zone, 0);
-    SysFreeString(display_uri);
     if(FAILED(hres)) {
         WARN("Could not map %s to zone: %08x\n", debugstr_w(display_uri), hres);
+        SysFreeString(display_uri);
         return TRUE;
     }

+    SysFreeString(display_uri);
     TRACE("zone %d\n", zone);
     return zone == URLZONE_UNTRUSTED;
 }