diff options
| author | Denys Vlasenko | 2017-05-22 19:34:45 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2017-05-22 19:34:45 +0200 |
| commit | f5470419404d643070db99d058405b714695b817 (patch) | |
| tree | 7ed502471976679437b86f2e2cb38c14984f8a1e /shell/ash.c | |
| parent | 0c63299b84513ff995413f820cb5f3d546c0c5d7 (diff) | |
| download | busybox-f5470419404d643070db99d058405b714695b817.zip busybox-f5470419404d643070db99d058405b714695b817.tar.gz | |
ash,hush: fix SIGCHLD interrupting read builtin
function old new delta
readcmd 169 217 +48
shell_builtin_read 1087 1097 +10
localcmd 366 364 -2
builtin_read 197 193 -4
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 2/2 up/down: 58/-6) Total: 52 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/ash.c')
| -rw-r--r-- | shell/ash.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/shell/ash.c b/shell/ash.c index 70ee15e..60c8ffe 100644 --- a/shell/ash.c +++ b/shell/ash.c @@ -13268,6 +13268,7 @@ readcmd(int argc UNUSED_PARAM, char **argv UNUSED_PARAM) /* "read -s" needs to save/restore termios, can't allow ^C * to jump out of it. */ + again: INT_OFF; r = shell_builtin_read(setvar0, argptr, @@ -13280,6 +13281,12 @@ readcmd(int argc UNUSED_PARAM, char **argv UNUSED_PARAM) ); INT_ON; + if ((uintptr_t)r == 1 && errno == EINTR) { + /* to get SIGCHLD: sleep 1 & read x; echo $x */ + if (pending_sig == 0) + goto again; + } + if ((uintptr_t)r > 1) ash_msg_and_raise_error(r); |