GpgFrontend/gpgme
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

* gpgmeplug.c (nextCertificate): Actually free the entire array

Browse Source 
and don't loop over tmp_dn and double free the first item.
Spotted by Bernhard Herzog.
This commit is contained in:
Werner Koch 2002-07-03 11:04:54 +00:00
parent ea0acccf2b
commit 992c88991d
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gpgmeplug/ChangeLog
View 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> 2002-07-01 Werner Koch <wk@gnupg.org>
* gpgmeplug.c (findCertificates): Reintroduced a free which must * gpgmeplug.c (findCertificates): Reintroduced a free which must

4
gpgmeplug/gpgmeplug.c
View File

@ -2125,8 +2125,8 @@ nextCertificate( struct CertIterator* it, struct CertificateInfo** result )
/*it->info.issuer = xstrdup(s);*/ /*it->info.issuer = xstrdup(s);*/
it->info.issuer = reorder_dn( issuer_dn ); it->info.issuer = reorder_dn( issuer_dn );
while( tmp_dn->key ) { while( tmp_dn->key ) {
free( issuer_dn->key ); free( tmp_dn->key );
free( issuer_dn->value ); free( tmp_dn->value );
++tmp_dn; ++tmp_dn;
} }
free( issuer_dn ); free( issuer_dn );