Re: [PATCHv2 1/6] pretty: free the gpg status buf
From: Jeff King <hidden>
Date: 2016-06-15 23:01:36
On Fri, Jun 13, 2014 at 12:42:43PM +0200, Michael J Gruber wrote:
quoted hunk ↗ jump to hunk
4a868fd (pretty: parse the gpg status lines rather than the output, 2013-02-14) made the gpg status lines available to callers and made sure they freed the used space, but missed one spot. Free the status line buffer also in the remaining spot. Signed-off-by: Michael J Gruber <redacted> --- pretty.c | 1 + 1 file changed, 1 insertion(+)diff --git a/pretty.c b/pretty.c index 4f51287..f1e8a70 100644 --- a/pretty.c +++ b/pretty.c@@ -1538,6 +1538,7 @@ void format_commit_message(const struct commit *commit, free(context.commit_encoding); logmsg_free(context.message, commit); free(context.signature_check.gpg_output); + free(context.signature_check.gpg_status); free(context.signature_check.signer);
What about .key? I would have expected your patch 6 to come first, which would fix this, and then save you from making similar mistakes in patch 3. :) -Peff