* 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 ae803c3349a0d1d5e3a86e011f721a3025c85749..185bdd249bdd490d3b14bf2407c3dc9e9afa1346 100644 (file)
--- 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 5e5df10826485d40e5b16fe6e26774a46862a6ce..72466a97acf4bc1a1324741e5402763265c0c1d2 100644 (file)
--- 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 );