diff options
author | Jakub Jelen <[email protected]> | 2021-04-12 04:06:17 +0000 |
---|---|---|
committer | NIIBE Yutaka <[email protected]> | 2021-04-12 04:06:17 +0000 |
commit | ad062b0a5b7d598081405ecfb71b51540281a1b7 (patch) | |
tree | fb9480faa66af5e99d4a3e713336036987e15b66 | |
parent | core: Fix gpgrt_wait_processes, by skipping invalid PID. (diff) | |
download | libgpg-error-ad062b0a5b7d598081405ecfb71b51540281a1b7.tar.gz libgpg-error-ad062b0a5b7d598081405ecfb71b51540281a1b7.zip |
build,tests: Fix leaks of memory or file pointer.
* src/mkheader.c (parse_config_h): Close FP.
* tests/t-b64.c (test_b64enc_string): Free STATE.
(test_b64dec_string): Free BUFFER.
* tests/t-syserror.c (main): Close FP.
--
GnuPG-bug-id: 5381
Signed-off-by: Jakub Jelen <[email protected]>
-rw-r--r-- | src/mkheader.c | 1 | ||||
-rw-r--r-- | tests/t-b64.c | 2 | ||||
-rw-r--r-- | tests/t-syserror.c | 1 |
3 files changed, 4 insertions, 0 deletions
diff --git a/src/mkheader.c b/src/mkheader.c index 77826da..1d2ea20 100644 --- a/src/mkheader.c +++ b/src/mkheader.c @@ -244,6 +244,7 @@ parse_config_h (const char *fname) { fprintf (stderr, "%s:%d: error reading file: %s\n", fname, lnr, strerror (errno)); + fclose (fp); return 1; } diff --git a/tests/t-b64.c b/tests/t-b64.c index 0171909..db08fc0 100644 --- a/tests/t-b64.c +++ b/tests/t-b64.c @@ -108,6 +108,7 @@ test_b64enc_string (const char *string, const char *expected, const char *title) err = gpgrt_b64enc_write (state, string, strlen (string)); if (err) { + free (state); fail ("gpgrt_b64enc_write failed: %s\n", gpg_strerror (err)); return err; } @@ -191,6 +192,7 @@ test_b64dec_string (const char *string, const char *expected, const char *title) gpgrt_log_debug_string (expected, "expect(len=%zu): ", strlen (expected)); } + free (buffer); return GPG_ERR_FALSE; } diff --git a/tests/t-syserror.c b/tests/t-syserror.c index a4cb983..93264dd 100644 --- a/tests/t-syserror.c +++ b/tests/t-syserror.c @@ -49,6 +49,7 @@ main (int argc, char *argv[]) } if (fp) { + fclose (fp); fprintf (stderr, "unable to run test\n"); return 1; } |