diff options
| author | Jakub Jelen <[email protected]> | 2021-05-20 08:13:51 +0000 |
|---|---|---|
| committer | Werner Koch <[email protected]> | 2021-05-20 11:38:07 +0000 |
| commit | a95ddffdcd58383cce93677be5e7e11c5c229a98 (patch) | |
| tree | 3705ec520ecc77efa85f07a85c66b47e9b162dca /agent/command.c | |
| parent | dirmngr: For KS_SEARCH return the fingerprint also with LDAP. (diff) | |
| download | gnupg-a95ddffdcd58383cce93677be5e7e11c5c229a98.tar.gz gnupg-a95ddffdcd58383cce93677be5e7e11c5c229a98.zip | |
agent: Avoid memory leaks in error code paths.
* agent/command.c (cmd_genkey): Use goto instead of return.
* agent/cvt-openpgp.c (convert_from_openpgp_main): Ditto.
* agent/genkey.c (agent_ask_new_passphrase): Fix typo to free correct
pointer
(agent_genkey): Release memory
* agent/gpg-agent.c (check_own_socket): Free sockname
* agent/protect-tool.c (read_key): Free buf.
(agent_askpin): Free passphrase
--
Signed-off-by: Jakub Jelen <[email protected]>
Changed original patch to not add a free before a GPG_ERR_BUG.
Signed-off-by: Werner Koch <[email protected]>
GnuPG-bug-id: 5393
Diffstat (limited to 'agent/command.c')
| -rw-r--r-- | agent/command.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/agent/command.c b/agent/command.c index 1f4fa9623..8ce7a15d3 100644 --- a/agent/command.c +++ b/agent/command.c @@ -1144,10 +1144,11 @@ cmd_genkey (assuan_context_t ctx, char *line) /* First inquire the parameters */ rc = print_assuan_status (ctx, "INQUIRE_MAXLEN", "%u", MAXLEN_KEYPARAM); - if (!rc) - rc = assuan_inquire (ctx, "KEYPARAM", &value, &valuelen, MAXLEN_KEYPARAM); if (rc) - return rc; + goto leave; + rc = assuan_inquire (ctx, "KEYPARAM", &value, &valuelen, MAXLEN_KEYPARAM); + if (rc) + goto leave; init_membuf (&outbuf, 512); |