summaryrefslogtreecommitdiff
path: root/builtin/commit.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2023-06-22 17:33:01 -0700
committerJunio C Hamano <gitster@pobox.com>2023-06-23 09:27:10 -0700
commit2b7b788fb31a74bcbff4e4c6efc6f3db6c3a49b7 (patch)
treeb24e68a77ce2af1e69d9c384189067e98b2cd614 /builtin/commit.c
parent6640c2d06d112675426cf436f0594f0e8c614848 (diff)
ll-merge: killing the external merge driver aborts the merge
When an external merge driver dies with a signal, we should not expect that the result left on the filesystem is in any useful state. However, because the current code uses the return value from run_command() and declares any positive value as a sign that the driver successfully left conflicts in the result, and because the return value from run_command() for a subprocess that died upon a signal is positive, we end up treating whatever garbage left on the filesystem as the result the merge driver wanted to leave us. run_command() returns larger than 128 (WTERMSIG(status) + 128, to be exact) when it notices that the subprocess died with a signal, so detect such a case and return LL_MERGE_ERROR from ll_ext_merge(). Signed-off-by: Junio C Hamano <gitster@pobox.com> Reviewed-by: Elijah Newren <newren@gmail.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions