From 2d0ab08470b2787ca70e41b64d71528fd3dfcba5 Mon Sep 17 00:00:00 2001 From: Phillip Wood Date: Sat, 22 Apr 2023 11:06:55 +0100 Subject: [PATCH] rebase -i: ignore signals when forking subprocesses If the user presses Ctrl-C to interrupt a program run by a rebase "exec" command then SIGINT will also be sent to the git process running the rebase resulting in it being killed. Fortunately the consequences of this are not severe as all the state necessary to continue the rebase is saved to disc but it would be better to avoid killing git and instead report that the command failed. A similar situation occurs when the sequencer runs "git commit" or "git merge". If the user generates SIGINT while editing the commit message then the git processes creating the commit will ignore it but the git process running the rebase will be killed. Fix this by ignoring SIGINT and SIGQUIT when forking "exec" commands, "git commit" and "git merge". This matches what git already does when running the user's editor and matches the behavior of the standard library's system() function. Signed-off-by: Phillip Wood --- sequencer.c | 19 +++++++++++++++++-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/sequencer.c b/sequencer.c index a66dcf8ab26360..26d70f68454562 100644 --- a/sequencer.c +++ b/sequencer.c @@ -1059,6 +1059,7 @@ static int run_git_commit(const char *defmsg, unsigned int flags) { struct child_process cmd = CHILD_PROCESS_INIT; + int res; if ((flags & CLEANUP_MSG) && (flags & VERBATIM_MSG)) BUG("CLEANUP_MSG and VERBATIM_MSG are mutually exclusive"); @@ -1116,10 +1117,16 @@ static int run_git_commit(const char *defmsg, if (!(flags & EDIT_MSG)) strvec_push(&cmd.args, "--allow-empty-message"); + sigchain_push(SIGINT, SIG_IGN); + sigchain_push(SIGQUIT, SIG_IGN); if (is_rebase_i(opts) && !(flags & EDIT_MSG)) - return run_command_silent_on_success(&cmd); + res = run_command_silent_on_success(&cmd); else - return run_command(&cmd); + res = run_command(&cmd); + sigchain_pop(SIGINT); + sigchain_pop(SIGQUIT); + + return res; } static int rest_is_empty(const struct strbuf *sb, int start) @@ -3628,10 +3635,14 @@ static int do_exec(struct repository *r, const char *command_line) struct child_process cmd = CHILD_PROCESS_INIT; int dirty, status; + sigchain_push(SIGINT, SIG_IGN); + sigchain_push(SIGQUIT, SIG_IGN); fprintf(stderr, _("Executing: %s\n"), command_line); cmd.use_shell = 1; strvec_push(&cmd.args, command_line); status = run_command(&cmd); + sigchain_pop(SIGINT); + sigchain_pop(SIGQUIT); /* force re-reading of the cache */ discard_index(r->index); @@ -4111,7 +4122,11 @@ static int do_merge(struct repository *r, NULL, 0); rollback_lock_file(&lock); + sigchain_push(SIGINT, SIG_IGN); + sigchain_push(SIGQUIT, SIG_IGN); ret = run_command(&cmd); + sigchain_pop(SIGINT); + sigchain_pop(SIGQUIT); /* force re-reading of the cache */ if (!ret) {