wbemprox: Fixed a memory leak in get_sd (Coverity).

e0905d78 · Marcus Meissner · Alexandre Julliard · 2c6cc0be · e0905d78
Commit e0905d78 authored Sep 21, 2014 by Marcus Meissner Committed by Alexandre Julliard Sep 22, 2014
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

security.c dlls/wbemprox/security.c +4 -1

No files found.
--- a/dlls/wbemprox/security.c
+++ b/dlls/wbemprox/security.c
@@ -120,8 +120,11 @@ static HRESULT get_sd( SECURITY_DESCRIPTOR **sd, DWORD *size )

    if (SUCCEEDED(hr))
    {
-        if (!MakeSelfRelativeSD(&absolute_sd, *sd, size))
+        if (!MakeSelfRelativeSD(&absolute_sd, *sd, size)) {
+            HeapFree( GetProcessHeap(), 0, *sd );
+	    *sd = NULL;
            hr = E_FAIL;
+        }
    }

    return hr;