[PATCH v3 0/1] MR423: ieframe fix

[David Kahurani (＠kahurani)]

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com>

--
  v3: ieframe: Fix memory leak in open_dde_url
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/423

[PATCH v3 1/1] ieframe: Fix memory leak in open_dde_url

[David Kahurani]

From: David Kahurani <k.kahurani(a)gmail.com>

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com>
---
 dlls/ieframe/iexplore.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/ieframe/iexplore.c b/dlls/ieframe/iexplore.c
index 1616d4d7f84..6eb86d93e5e 100644
--- a/dlls/ieframe/iexplore.c
+++ b/dlls/ieframe/iexplore.c
@@ -1027,10 +1027,12 @@ static HDDEDATA open_dde_url(WCHAR *dde_url)
     }
 
     hres = IWebBrowser2_Navigate2(&ie->IWebBrowser2_iface, &urlv, NULL, NULL, NULL, NULL);
-    if(FAILED(hres))
-        return 0;
 
+    SysFreeString(V_BSTR(&urlv));
     IWebBrowser2_Release(&ie->IWebBrowser2_iface);
+
+    if(FAILED(hres))
+        return 0;
     return ULongToHandle(DDE_FACK);
 }
 
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/423

[PATCH v3 0/1] MR423: ieframe fix - approved

[Jacek Caban (＠jacek)]

This merge request was approved by Jacek Caban.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/423