Juan Lang : crypt32: Only free allocated handles.

[Alexandre Julliard]

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

Author: Juan Lang 
Date:   Tue Aug 21 15:13:05 2007 -0700

crypt32: Only free allocated handles.

---

dlls/crypt32/msg.c |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/dlls/crypt32/msg.c b/dlls/crypt32/msg.c
index 4c2c617..2107332 100644
--- a/dlls/crypt32/msg.c
+++ b/dlls/crypt32/msg.c
@@ -831,9 +831,12 @@ static void CSignedMsgData_CloseHandles(CSignedMsgData *msg_data)

for (i = 0; i < msg_data->info->cSignerInfo; i++)
     {
-        CryptDestroyKey(msg_data->signerHandles[i].key);
-        CryptDestroyHash(msg_data->signerHandles[i].contentHash);
-        CryptDestroyHash(msg_data->signerHandles[i].authAttrHash);
+        if (msg_data->signerHandles[i].key)
+            CryptDestroyKey(msg_data->signerHandles[i].key);
+        if (msg_data->signerHandles[i].contentHash)
+            CryptDestroyHash(msg_data->signerHandles[i].contentHash);
+        if (msg_data->signerHandles[i].authAttrHash)
+            CryptDestroyHash(msg_data->signerHandles[i].authAttrHash);
     }
     CryptMemFree(msg_data->signerHandles);
 }