crypt32(13/13): Eliminate a double free in the tests

View: New views
1 Messages — Rating Filter:   Alert me  

crypt32(13/13): Eliminate a double free in the tests

by Juan Lang-4 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

CertDeleteCertificatefromStore already frees a context, so freeing it
again is a mistake.
--Juan

[0013-Eliminate-a-double-free-in-the-tests.patch]

From 476f1afeb9c89a943144c93de0051d308574bb4f Mon Sep 17 00:00:00 2001
From: Juan Lang <juan.lang@...>
Date: Tue, 3 Nov 2009 17:22:10 -0800
Subject: [PATCH] Eliminate a double free in the tests

---
 dlls/crypt32/tests/store.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/dlls/crypt32/tests/store.c b/dlls/crypt32/tests/store.c
index 419d2dd..5c019fa 100644
--- a/dlls/crypt32/tests/store.c
+++ b/dlls/crypt32/tests/store.c
@@ -2096,7 +2096,6 @@ static void test_I_UpdateStore(void)
     certs = countCertsInStore(store1);
     ok(certs == 0, "Expected 0 certs, got %d\n", certs);
 
-    CertFreeCertificateContext(cert);
     CertCloseStore(store1, 0);
     CertCloseStore(store2, 0);
 }
--
1.6.0.6