From: Werner Koch <wk@gnupg.org>
Date: Wed, 3 Jul 2002 11:04:54 +0000 (+0000)
Subject: * gpgmeplug.c (nextCertificate): Actually free the entire array
X-Git-Tag: gpgme-0-3-9~41
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=992c88991d2d97327626a95c712477254450b75c;p=gpgme.git

* gpgmeplug.c (nextCertificate): Actually free the entire array
and don't loop over tmp_dn and double free the first item.
Spotted by Bernhard Herzog.
---

diff --git a/gpgmeplug/ChangeLog b/gpgmeplug/ChangeLog
index ae803c3..185bdd2 100644
--- a/gpgmeplug/ChangeLog
+++ b/gpgmeplug/ChangeLog
@@ -1,3 +1,9 @@
+2002-07-03  Werner Koch  <wk@gnupg.org>
+
+	* gpgmeplug.c (nextCertificate): Actually free the entire array
+	and don't loop over tmp_dn and double free the first item.
+	Spotted by Bernhard Herzog.
+
 2002-07-01  Werner Koch  <wk@gnupg.org>
 
 	* gpgmeplug.c (findCertificates): Reintroduced a free which must
diff --git a/gpgmeplug/gpgmeplug.c b/gpgmeplug/gpgmeplug.c
index 5e5df10..72466a9 100644
--- a/gpgmeplug/gpgmeplug.c
+++ b/gpgmeplug/gpgmeplug.c
@@ -2125,8 +2125,8 @@ nextCertificate( struct CertIterator* it, struct CertificateInfo** result )
       /*it->info.issuer = xstrdup(s);*/
       it->info.issuer = reorder_dn( issuer_dn );
       while( tmp_dn->key ) {
-	free( issuer_dn->key );
-	free( issuer_dn->value );
+	free( tmp_dn->key );
+	free( tmp_dn->value );
 	++tmp_dn;
       }
       free( issuer_dn );