mirror of
https://github.com/sheumann/hush.git
synced 2024-12-27 01:32:08 +00:00
hush: do not print killer signal's name for SIGPIPE
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
parent
b9348440b0
commit
7c6f2468cc
@ -3892,6 +3892,7 @@ sprint_status(char *s, int status, int sigonly)
|
||||
#endif
|
||||
}
|
||||
st &= 0x7f;
|
||||
//TODO: use bbox's get_signame? strsignal adds ~600 bytes to text+rodata
|
||||
col = fmtstr(s, 32, strsignal(st));
|
||||
if (WCOREDUMP(status)) {
|
||||
col += fmtstr(s + col, 16, " (core dumped)");
|
||||
|
@ -6504,13 +6504,15 @@ static int checkjobs(struct pipe *fg_pipe)
|
||||
fg_pipe->alive_cmds--;
|
||||
ex = WEXITSTATUS(status);
|
||||
/* bash prints killer signal's name for *last*
|
||||
* process in pipe (prints just newline for SIGINT).
|
||||
* process in pipe (prints just newline for SIGINT/SIGPIPE).
|
||||
* Mimic this. Example: "sleep 5" + (^\ or kill -QUIT)
|
||||
*/
|
||||
if (WIFSIGNALED(status)) {
|
||||
int sig = WTERMSIG(status);
|
||||
if (i == fg_pipe->num_cmds-1)
|
||||
printf("%s\n", sig == SIGINT ? "" : get_signame(sig));
|
||||
/* TODO: use strsignal() instead for bash compat? but that's bloat... */
|
||||
printf("%s\n", 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? */
|
||||
ex = sig + 128;
|
||||
|
Loading…
Reference in New Issue
Block a user