* gpgmeplug.c (nextCertificate): Actually free the entire array
authorWerner Koch <wk@gnupg.org>
Wed, 3 Jul 2002 11:04:54 +0000 (11:04 +0000)
committerWerner Koch <wk@gnupg.org>
Wed, 3 Jul 2002 11:04:54 +0000 (11:04 +0000)
and don't loop over tmp_dn and double free the first item.
Spotted by Bernhard Herzog.

gpgmeplug/ChangeLog
gpgmeplug/gpgmeplug.c

index ae803c3..185bdd2 100644 (file)
@@ -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
index 5e5df10..72466a9 100644 (file)
@@ -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 );