summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLibravatar Stefan Beller <sbeller@google.com>2018-06-01 13:01:45 -0700
committerLibravatar Junio C Hamano <gitster@pobox.com>2018-06-04 11:36:35 +0900
commit19517fb964f81c1bd3ddd006e0fc0119167d9cb2 (patch)
tree020be69759e8b883a0c90c0e8051955aff24303c
parentsubmodule--helper: plug mem leak in print_default_remote (diff)
downloadtgif-19517fb964f81c1bd3ddd006e0fc0119167d9cb2.tar.xz
sequencer.c: plug leaks in do_pick_commit
Going to leave, we additionally free the author and commit message and make sure to call update_abort_safety_file(). Signed-off-by: Stefan Beller <sbeller@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--sequencer.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/sequencer.c b/sequencer.c
index 72b4d8ecae..ae6f2c27ec 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -1772,7 +1772,8 @@ static int do_pick_commit(enum todo_command command, struct commit *commit,
res = do_recursive_merge(base, next, base_label, next_label,
&head, &msgbuf, opts);
if (res < 0)
- return res;
+ goto leave;
+
res |= write_message(msgbuf.buf, msgbuf.len,
git_path_merge_msg(), 0);
} else {