[PATCH v5 10/36] sequencer: trivial fix
STALE3761d
From: Felipe Contreras <hidden>
Date: 2016-06-15 22:57:39
Subsystem:
the rest · Maintainer:
Linus Torvalds
Revision v5 of 3 in this series.
Revisions (3)
- v4 [diff vs current]
- v5 current
- v6 [diff vs current]
We should free objects before leaving. Signed-off-by: Felipe Contreras <redacted> --- builtin/sequencer.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/builtin/sequencer.c b/builtin/sequencer.c
index b2c8c94..23b01b7 100644
--- a/builtin/sequencer.c
+++ b/builtin/sequencer.c@@ -539,8 +539,10 @@ static int do_pick_commit(struct commit *commit, struct replay_opts *opts) } 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);
--
1.8.3.698.g079b096