diff options
author | zhangguangzhi <[email protected]> | 2023-05-18 12:24:41 +0000 |
---|---|---|
committer | NIIBE Yutaka <[email protected]> | 2023-06-20 00:21:10 +0000 |
commit | 3efd0052854d374d0900124a54fccbc7f1257242 (patch) | |
tree | 461682e73963ea738c35219ead263285bafe27f1 | |
parent | delete redundant characters (diff) | |
download | gnupg-3efd0052854d374d0900124a54fccbc7f1257242.tar.gz gnupg-3efd0052854d374d0900124a54fccbc7f1257242.zip |
kbx: Close file handle when return.
* kbx/keybox-dump.c (_keybox_dump_find_dups): Close FP on the error
paths.
--
GnuPG-bug-id: 6495
Signed-off-by: zhangguangzhi <[email protected]>
-rw-r--r-- | kbx/keybox-dump.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/kbx/keybox-dump.c b/kbx/keybox-dump.c index 3e66b72a1..3bf63c98b 100644 --- a/kbx/keybox-dump.c +++ b/kbx/keybox-dump.c @@ -810,6 +810,8 @@ _keybox_dump_find_dups (const char *filename, int print_them, FILE *outfp) gpg_error_t tmperr = gpg_error_from_syserror (); fprintf (outfp, "error allocating array for '%s': %s\n", filename, strerror(errno)); + if (fp != es_stdin) + es_fclose (fp); return tmperr; } dupitems_count = 0; @@ -834,6 +836,8 @@ _keybox_dump_find_dups (const char *filename, int print_them, FILE *outfp) fprintf (outfp, "error reallocating array for '%s': %s\n", filename, strerror(errno)); free (dupitems); + if (fp != es_stdin) + es_fclose (fp); return tmperr; } dupitems = tmp; |