From b65d6cb00fa0ea51bac4c4e62b576b43ae2c996b Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Sat, 24 Oct 2020 03:33:32 +0200 Subject: hush: do not print killing signal name in `cmd_whihc_dies_on_signal` function old new delta process_wait_result 438 449 +11 Signed-off-by: Denys Vlasenko --- shell/hush.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'shell/hush.c') diff --git a/shell/hush.c b/shell/hush.c index e9cec1cc9..2b34b7da5 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -8698,9 +8698,15 @@ static int process_wait_result(struct pipe *fg_pipe, pid_t childpid, int status) */ if (WIFSIGNALED(status)) { int sig = WTERMSIG(status); - if (i == fg_pipe->num_cmds-1) + if (G.run_list_level == 1 + /* ^^^^^ Do not print in nested contexts, example: + * echo `sleep 1; sh -c 'kill -9 $$'` - prints "137", NOT "Killed 137" + */ + && i == fg_pipe->num_cmds-1 + ) { /* TODO: use strsignal() instead for bash compat? but that's bloat... */ puts(sig == SIGINT || sig == SIGPIPE ? "" : get_signame(sig)); + } /* TODO: if (WCOREDUMP(status)) + " (core dumped)"; */ /* TODO: MIPS has 128 sigs (1..128), what if sig==128 here? * Maybe we need to use sig | 128? */ -- cgit v1.2.3