From c42cfc460aa56ecdd311b952262d0ce4d0c1962e Mon Sep 17 00:00:00 2001 From: Patrick McCarty Date: Wed, 10 Jan 2018 16:24:19 -0800 Subject: [PATCH] Fix memleak in signature.c Also, initialize the existing pointers that *are* freed, just in case. Signed-off-by: Patrick McCarty --- src/signature.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/signature.c b/src/signature.c index a04dd906..fc0d294b 100644 --- a/src/signature.c +++ b/src/signature.c @@ -418,9 +418,9 @@ int verify_callback(int ok, X509_STORE_CTX *stor) */ bool download_and_verify_signature(const char *data_url, const char *data_filename, int version, bool mix_exists) { - char *local; - char *sig_url; - char *sig_filename; + char *local = NULL; + char *sig_url = NULL; + char *sig_filename = NULL; int ret; bool result; @@ -452,6 +452,7 @@ bool download_and_verify_signature(const char *data_url, const char *data_filena result = false; } out: + free(local); free(sig_filename); free(sig_url); return result;