diff options
author | Denis Vlasenko | 2007-03-21 20:17:27 +0000 |
---|---|---|
committer | Denis Vlasenko | 2007-03-21 20:17:27 +0000 |
commit | 7c139b477882a06a86c31928d997d43c696dbc62 (patch) | |
tree | ca1e64745da7ff43f154ad6ea66c37f9e97840fb | |
parent | b8e72fdde1186f41963697d57383c08402513ca8 (diff) | |
download | busybox-7c139b477882a06a86c31928d997d43c696dbc62.zip busybox-7c139b477882a06a86c31928d997d43c696dbc62.tar.gz |
ash: fix signal handling
-rw-r--r-- | shell/ash.c | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/shell/ash.c b/shell/ash.c index 29156c1..a5ffaaf 100644 --- a/shell/ash.c +++ b/shell/ash.c @@ -183,7 +183,7 @@ static volatile sig_atomic_t intpending; /* do we generate EXSIG events */ static int exsig; /* last pending signal */ -static volatile sig_atomic_t pendingsigs; +static volatile sig_atomic_t pendingsig; /* * Sigmode records the current value of the signal handlers for the various @@ -239,8 +239,15 @@ static void raise_interrupt(void) { int i; + sigset_t mask; intpending = 0; + /* Signal is not automatically re-enabled after it is raised, + * do it ourself */ + sigemptyset(&mask); + sigprocmask(SIG_SETMASK, &mask, 0); + /* pendingsig = 0; - now done in onsig() */ + i = EXSIG; if (gotsig[SIGINT - 1] && !trap[SIGINT]) { if (!(rootshell && iflag)) { @@ -300,7 +307,7 @@ force_int_on(void) do { \ exsig++; \ xbarrier(); \ - if (pendingsigs) \ + if (pendingsig) \ raise_exception(EXSIG); \ } while (0) /* EXSIG is turned off by evalbltin(). */ @@ -324,11 +331,13 @@ static void onsig(int signo) { gotsig[signo - 1] = 1; - pendingsigs = signo; + pendingsig = signo; if (exsig || (signo == SIGINT && !trap[SIGINT])) { - if (!suppressint) + if (!suppressint) { + pendingsig = 0; raise_interrupt(); + } intpending = 1; } } @@ -7441,7 +7450,7 @@ dotrap(void) int skip = 0; savestatus = exitstatus; - pendingsigs = 0; + pendingsig = 0; xbarrier(); for (i = 0, q = gotsig; i < NSIG - 1; i++, q++) { @@ -7580,7 +7589,7 @@ evaltree(union node *n, int flags) out: if ((checkexit & exitstatus)) evalskip |= SKIPEVAL; - else if (pendingsigs && dotrap()) + else if (pendingsig && dotrap()) goto exexit; if (flags & EV_EXIT) { @@ -8447,7 +8456,7 @@ evalcommand(union node *cmd, int flags) if (i == EXINT) j = SIGINT; if (i == EXSIG) - j = pendingsigs; + j = pendingsig; if (j) exit_status = j + 128; exitstatus = exit_status; |