From 741b0f37f8c274cd3d4206e8ddf9d3aee74f66c1 Mon Sep 17 00:00:00 2001
From: Hans Leidekker <hans@codeweavers.com>
Date: Thu, 14 Apr 2011 14:40:15 +0200
Subject: [PATCH] msi: Fix a memory leak (valgrind).

---
 dlls/msi/msi.c | 25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/dlls/msi/msi.c b/dlls/msi/msi.c
index def9e72eb32..4a79daf14b1 100644
--- a/dlls/msi/msi.c
+++ b/dlls/msi/msi.c
@@ -2395,25 +2395,36 @@ HRESULT WINAPI MsiGetFileSignatureInformationW( LPCWSTR path, DWORD flags, PCCER
     data.dwProvFlags         = 0;
     data.dwUIContext         = WTD_UICONTEXT_INSTALL;
     hr = WinVerifyTrustEx( INVALID_HANDLE_VALUE, &generic_verify_v2, &data );
-    if (FAILED(hr)) return hr;
+    if (FAILED(hr)) goto done;
 
-    signer = WTHelperGetProvSignerFromChain( data.hWVTStateData, 0, FALSE, 0 );
-    if (!signer) return TRUST_E_NOSIGNATURE;
+    if (!(signer = WTHelperGetProvSignerFromChain( data.hWVTStateData, 0, FALSE, 0 )))
+    {
+        hr = TRUST_E_NOSIGNATURE;
+        goto done;
+    }
     if (hash)
     {
         DWORD len = signer->psSigner->EncryptedHash.cbData;
         if (*hashlen < len)
         {
             *hashlen = len;
-            return HRESULT_FROM_WIN32(ERROR_MORE_DATA);
+            hr = HRESULT_FROM_WIN32(ERROR_MORE_DATA);
+            goto done;
         }
         memcpy( hash, signer->psSigner->EncryptedHash.pbData, len );
         *hashlen = len;
     }
-    provider = WTHelperGetProvCertFromChain( signer, 0 );
-    if (!provider) return TRUST_E_PROVIDER_UNKNOWN;
+    if (!(provider = WTHelperGetProvCertFromChain( signer, 0 )))
+    {
+        hr = TRUST_E_PROVIDER_UNKNOWN;
+        goto done;
+    }
     *cert = CertDuplicateCertificateContext( provider->pCert );
-    return S_OK;
+
+done:
+    data.dwStateAction = WTD_STATEACTION_CLOSE;
+    WinVerifyTrustEx( INVALID_HANDLE_VALUE, &generic_verify_v2, &data );
+    return hr;
 }
 
 /******************************************************************