emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

acpid: do not install handlers for fatal signals

Browse Source 
For acpid, it's ok to die at once.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2012-02-05 15:08:08 +01:00
parent 6e9d047c15
commit bbf1e3c144
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
util-linux/acpid.c
View File

@ -268,8 +268,12 @@ int acpid_main(int argc UNUSED_PARAM, char **argv)
xchdir(opt_dir); xchdir(opt_dir);
/* We spawn children but don't wait for them. Prevent zombies: */
bb_signals((1 << SIGCHLD), SIG_IGN); bb_signals((1 << SIGCHLD), SIG_IGN);
bb_signals(BB_FATAL_SIGS, record_signo); // If you enable this, (1) explain why, (2)
// make sure while(poll) loop below is still interruptible
// by SIGTERM et al:
//bb_signals(BB_FATAL_SIGS, record_signo);
pfd = NULL; pfd = NULL;
nfd = 0; nfd = 0;
@ -337,7 +341,7 @@ int acpid_main(int argc UNUSED_PARAM, char **argv)
} }
if (!event) if (!event)
continue; continue;
// spawn event handler /* spawn event handler */
process_event(event); process_event(event);
} }
} }