From: Felipe Contreras Date: Mon, 27 May 2013 16:52:18 GMT Subject: [PATCH 1/2] sequencer: trivial fix Message-ID: <1369673539-28692-2-git-send-email-felipe.contreras@gmail.com> In-Reply-To: <1369673539-28692-1-git-send-email-felipe.contreras@gmail.com> We should free objects before leaving. Signed-off-by: Felipe Contreras --- sequencer.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/sequencer.c b/sequencer.c index ab6f8a7..7eeae2f 100644 --- a/sequencer.c +++ b/sequencer.c @@ -626,12 +626,15 @@ static int do_pick_commit(struct commit *commit, struct replay_opts *opts) rerere(opts->allow_rerere_auto); } else { int allow = allow_empty(opts, commit); - if (allow < 0) - return allow; + if (allow < 0) { + res = allow; + goto leave; + } if (!opts->no_commit) res = run_git_commit(defmsg, opts, allow); } +leave: free_message(&msg); free(defmsg); -- 1.8.3.rc3.312.g47657de