[PATCH 1/3] sequencer: trivial fix
From: Felipe Contreras <hidden>
Date: 2016-06-15 22:57:35
Subsystem:
the rest · Maintainer:
Linus Torvalds
We should free objects before leaving. Signed-off-by: Felipe Contreras <redacted> --- 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.698.g079b096