diff options
author | Jakub Jelen <[email protected]> | 2021-04-12 18:05:48 +0000 |
---|---|---|
committer | Werner Koch <[email protected]> | 2021-05-20 12:42:29 +0000 |
commit | fa0771f609b5fcb104c64a11aefc501b7a91696d (patch) | |
tree | 6e2929ab9eb917f4d2c9e2f44f456bcee06de838 /g10/keyedit.c | |
parent | dirmgr: Avoid double free (diff) | |
download | gnupg-fa0771f609b5fcb104c64a11aefc501b7a91696d.tar.gz gnupg-fa0771f609b5fcb104c64a11aefc501b7a91696d.zip |
g10: Avoid memory leaks
* g10/call-agent.c (card_keyinfo_cb): free keyinfo. Restructure to
avoid backward gotos.
* g10/keyedit.c (menu_set_keyserver_url): properly enclose the block
* g10/keygen.c (gen_card_key): free pk and pkt
--
Signed-off-by: Jakub Jelen <[email protected]>
GnuPG-bug-id: 5393
Additional changes:
- Restructure to avoid backward gotos.
Signed-off-by: Werner Koch <[email protected]>
Diffstat (limited to 'g10/keyedit.c')
-rw-r--r-- | g10/keyedit.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/g10/keyedit.c b/g10/keyedit.c index 7255676e7..122548df5 100644 --- a/g10/keyedit.c +++ b/g10/keyedit.c @@ -5382,14 +5382,16 @@ menu_set_keyserver_url (ctrl_t ctrl, const char *url, kbnode_t pub_keyblock) uri ? _("Are you sure you want to replace it? (y/N) ") : _("Are you sure you want to delete it? (y/N) "))) - xfree (user); - continue; + { + xfree (user); + continue; + } } else if (uri == NULL) { /* There is no current keyserver URL, so there is no point in trying to un-set it. */ - xfree (user); + xfree (user); continue; } |