Nikolay Sivov : mshtml: Fix BSTR leak in SetReferrerWithPolicy() (Valgrind ).

Alexandre Julliard julliard at winehq.org
Mon Nov 14 17:07:08 CST 2016


Module: wine
Branch: master
Commit: 15a07c0725f0a2aa165d4ecc4b67633933563e85
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=15a07c0725f0a2aa165d4ecc4b67633933563e85

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Thu Nov 10 00:21:56 2016 +0300

mshtml: Fix BSTR leak in SetReferrerWithPolicy() (Valgrind).

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Jacek Caban<jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

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

diff --git a/dlls/mshtml/nsio.c b/dlls/mshtml/nsio.c
index 0e2a748..ba21312 100644
--- a/dlls/mshtml/nsio.c
+++ b/dlls/mshtml/nsio.c
@@ -1343,8 +1343,10 @@ static nsresult NSAPI nsChannel_SetReferrerWithPolicy(nsIHttpChannel *iface, nsI
     }
 
     hres = IUri_GetDisplayUri(referrer->uri, &referrer_uri);
-    if(SUCCEEDED(hres) )
+    if(SUCCEEDED(hres)) {
         set_http_header(&This->request_headers, refererW, sizeof(refererW)/sizeof(WCHAR), referrer_uri, SysStringLen(referrer_uri));
+        SysFreeString(referrer_uri);
+    }
 
     This->referrer = (nsIURI*)&referrer->nsIFileURL_iface;
     return NS_OK;




More information about the wine-cvs mailing list