ole32: Fix a memory leak.

caa9f89a · Rob Shearman · Alexandre Julliard · b6553e7e · caa9f89a
Commit caa9f89a authored Jan 07, 2008 by Rob Shearman Committed by Alexandre Julliard Jan 07, 2008
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

oleproxy.c dlls/ole32/oleproxy.c +5 -3

No files found.
--- a/dlls/ole32/oleproxy.c
+++ b/dlls/ole32/oleproxy.c
@@ -574,9 +574,11 @@ static HRESULT WINAPI RemUnkStub_Invoke(LPRPCSTUBBUFFER iface,
    *(HRESULT *)buf = hr;
    buf += sizeof(HRESULT);
    
-    if (hr) return hr;
-    /* FIXME: pQIResults is a unique pointer so pQIResults can be NULL! */
-    memcpy(buf, pQIResults, cIids * sizeof(REMQIRESULT));
+    if (hr == S_OK)
+      /* FIXME: pQIResults is a unique pointer so pQIResults can be NULL! */
+      memcpy(buf, pQIResults, cIids * sizeof(REMQIRESULT));
+
+    CoTaskMemFree(pQIResults);

    break;
  }