diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2021-06-05 16:20:05 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2021-06-05 16:20:05 +0200 |
commit | 5dadd497ffd9835a2860cf89ad781d1b513803dc (patch) | |
tree | 7ad8eb72b3cd1b5a3d94cd6d2ead9e86f2779c0a /libbb | |
parent | d3e1090308b6d3c55e01a2000a743b73605ddd7f (diff) | |
download | busybox-w32-5dadd497ffd9835a2860cf89ad781d1b513803dc.tar.gz busybox-w32-5dadd497ffd9835a2860cf89ad781d1b513803dc.tar.bz2 busybox-w32-5dadd497ffd9835a2860cf89ad781d1b513803dc.zip |
runsv: robustify signal handling - SIGTERM to child between vfork and exec could mess things up
While at it, rename bb_signals_recursive_norestart() to bb_signals_norestart():
"recursive" was implying we are setting SA_NODEFER allowing signal handler
to be entered recursively, but we do not do that.
function old new delta
bb_signals_norestart - 70 +70
startservice 380 394 +14
bb_signals_recursive_norestart 70 - -70
------------------------------------------------------------------------------
(add/remove: 1/1 grow/shrink: 1/0 up/down: 84/-70) Total: 14 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libbb')
-rw-r--r-- | libbb/signals.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libbb/signals.c b/libbb/signals.c index d3d84ef6a..0bebc847d 100644 --- a/libbb/signals.c +++ b/libbb/signals.c | |||
@@ -56,7 +56,7 @@ void FAST_FUNC bb_signals(int sigs, void (*f)(int)) | |||
56 | } | 56 | } |
57 | } | 57 | } |
58 | 58 | ||
59 | void FAST_FUNC bb_signals_recursive_norestart(int sigs, void (*f)(int)) | 59 | void FAST_FUNC bb_signals_norestart(int sigs, void (*f)(int)) |
60 | { | 60 | { |
61 | int sig_no = 0; | 61 | int sig_no = 0; |
62 | int bit = 1; | 62 | int bit = 1; |