From 992c88991d2d97327626a95c712477254450b75c Mon Sep 17 00:00:00 2001 From: Werner Koch Date: Wed, 3 Jul 2002 11:04:54 +0000 Subject: [PATCH] * 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. --- gpgmeplug/ChangeLog | 6 ++++++ gpgmeplug/gpgmeplug.c | 4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) 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 + + * 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 * 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 ); -- 2.26.2