summaryrefslogtreecommitdiff
path: root/procps
diff options
context:
space:
mode:
authorDenis Vlasenko2006-09-27 14:25:33 +0000
committerDenis Vlasenko2006-09-27 14:25:33 +0000
commit0bb628f4f3fbd0fc0a96d257830fcbbb14db2bd1 (patch)
treefd0cff9651bdffc5f4b8ed5bfaee7b2671384e08 /procps
parenta77947f5bb9ef38b0f1fc6a2a5b500fa6ade601e (diff)
downloadbusybox-0bb628f4f3fbd0fc0a96d257830fcbbb14db2bd1.zip
busybox-0bb628f4f3fbd0fc0a96d257830fcbbb14db2bd1.tar.gz
kill: implement killall5. OpenWRT folks will be happy.
However their code was unusably different from sysvinit original. Run tested.
Diffstat (limited to 'procps')
-rw-r--r--procps/Config.in5
-rw-r--r--procps/kill.c76
2 files changed, 53 insertions, 28 deletions
diff --git a/procps/Config.in b/procps/Config.in
index 2183fab..2d4ad3b 100644
--- a/procps/Config.in
+++ b/procps/Config.in
@@ -38,6 +38,11 @@ config CONFIG_KILLALL
specified commands. If no signal name is specified, SIGTERM is
sent.
+config CONFIG_KILLALL5
+ bool "killall5"
+ default n
+ depends on CONFIG_KILL
+
config CONFIG_PIDOF
bool "pidof"
default n
diff --git a/procps/kill.c b/procps/kill.c
index 654467a..fe9b242 100644
--- a/procps/kill.c
+++ b/procps/kill.c
@@ -13,17 +13,18 @@
int kill_main(int argc, char **argv)
{
char *arg;
- int killall, signo = SIGTERM, errors = 0, quiet = 0;
-
- killall = (ENABLE_KILLALL && bb_applet_name[4]=='a') ? 1 : 0;
+ pid_t pid;
+ int signo = SIGTERM, errors = 0, quiet = 0;
+ const int killall = (ENABLE_KILLALL && bb_applet_name[4]=='a'
+ && (!ENABLE_KILLALL5 || bb_applet_name[7]!='5'));
+ const int killall5 = (ENABLE_KILLALL5 && bb_applet_name[4]=='a'
+ && (!ENABLE_KILLALL || bb_applet_name[7]=='5'));
/* Parse any options */
argc--;
arg = *++argv;
- if (argc<1)
- bb_show_usage();
- if (arg[0]!='-') {
+ if (argc<1 || arg[0]!='-') {
goto do_it_now;
}
@@ -79,42 +80,48 @@ int kill_main(int argc, char **argv)
do_it_now:
- /* Pid or name required */
- if (argc<1)
- bb_show_usage();
-
- if (!killall) {
- /* Looks like they want to do a kill. Do that */
- while (arg) {
- int pid;
-
- if (!isdigit(arg[0]) && arg[0]!='-')
- bb_error_msg_and_die("bad pid '%s'", arg);
- pid = strtol(arg, NULL, 0);
- if (kill(pid, signo)!=0) {
- bb_perror_msg("cannot kill pid %d", pid);
- errors++;
- }
- arg = *++argv;
+ if (killall5) {
+ pid_t sid;
+ procps_status_t* p;
+
+ /* kill(-1, sig) on Linux (at least 2.1.x)
+ * might send signal to the calling process too */
+ signal(SIGTERM, SIG_IGN);
+ /* Now stop all processes */
+ kill(-1, SIGSTOP);
+ /* Find out our own session id */
+ pid = getpid();
+ sid = getsid(pid);
+ /* Now kill all processes except our session */
+ while ((p = procps_scan(0))!=0) {
+ if (getsid(p->pid)!=sid && p->pid!=pid && p->pid!=1)
+ kill(p->pid, signo);
}
+ /* And let them continue */
+ kill(-1, SIGCONT);
+ return 0;
+ }
- } else {
- pid_t myPid = getpid();
+ /* Pid or name required for kill/killall */
+ if (argc<1)
+ bb_show_usage();
+ if (killall) {
/* Looks like they want to do a killall. Do that */
+ pid = getpid();
while (arg) {
long* pidList;
pidList = find_pid_by_name(arg);
if (!pidList || *pidList<=0) {
errors++;
- if (quiet==0)
+ if (!quiet)
bb_error_msg("%s: no process killed", arg);
} else {
long *pl;
- for (pl = pidList; *pl!=0 ; pl++) {
- if (*pl==myPid)
+ for (pl = pidList; *pl!=0; pl++) {
+ if (*pl==pid)
continue;
if (kill(*pl, signo)!=0) {
errors++;
@@ -126,7 +133,20 @@ do_it_now:
free(pidList);
arg = *++argv;
}
+ return errors;
}
+ /* Looks like they want to do a kill. Do that */
+ while (arg) {
+ if (!isdigit(arg[0]) && arg[0]!='-')
+ bb_error_msg_and_die("bad pid '%s'", arg);
+ pid = strtol(arg, NULL, 0);
+ /* FIXME: better overflow check? */
+ if (kill(pid, signo)!=0) {
+ bb_perror_msg("cannot kill pid %ld", (long)pid);
+ errors++;
+ }
+ arg = *++argv;
+ }
return errors;
}