inetcomm: Fixed buffer leak on error path (Coverity).

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

inetcomm: Fixed buffer leak on error path (Coverity).
dcc545be · Nikolay Sivov · Alexandre Julliard · b1d46e0b · dcc545be
Commit dcc545be authored Apr 30, 2017 by Nikolay Sivov Committed by Alexandre Julliard May 01, 2017
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

protocol.c dlls/inetcomm/protocol.c +3 -1

No files found.
--- a/dlls/inetcomm/protocol.c
+++ b/dlls/inetcomm/protocol.c
@@ -495,8 +495,10 @@ static HRESULT WINAPI MimeHtmlProtocol_Start(IInternetProtocol *iface, const WCH
    binding->url[url.mhtml_len] = 0;

    hres = CreateURLMoniker(NULL, binding->url, &mon);
-    if(FAILED(hres))
+    if(FAILED(hres)) {
+        heap_free(binding);
        return hres;
+    }

    binding->IBindStatusCallback_iface.lpVtbl = &BindStatusCallbackVtbl;
    binding->ref = 1;