diff options
| author | Lennart Poettering <lennart@poettering.net> | 2011-07-12 04:05:33 +0200 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2011-07-12 04:05:33 +0200 |
| commit | 3611581ebdabbe3a1d6a9b5310a0b59792279d7d (patch) | |
| tree | 3652d8d0764fdd90149eb1ed03bd1b8f92d2382c /src/socket.c | |
| parent | ee17ee7c5cc22c286f943578918beaa78bbc86db (diff) | |
service: properly handle who argument on D-Bus kill calls
Diffstat (limited to 'src/socket.c')
| -rw-r--r-- | src/socket.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/socket.c b/src/socket.c index 8b78a3fd2..3a7aa9403 100644 --- a/src/socket.c +++ b/src/socket.c @@ -2030,11 +2030,12 @@ static int socket_kill(Unit *u, KillWho who, KillMode mode, int signo, DBusError return -ENOENT; } - if (s->control_pid > 0) - if (kill(s->control_pid, signo) < 0) - r = -errno; + if (who == KILL_CONTROL || who == KILL_ALL) + if (s->control_pid > 0) + if (kill(s->control_pid, signo) < 0) + r = -errno; - if (mode == KILL_CONTROL_GROUP) { + if (who == KILL_ALL && mode == KILL_CONTROL_GROUP) { int q; if (!(pid_set = set_new(trivial_hash_func, trivial_compare_func))) @@ -2048,7 +2049,7 @@ static int socket_kill(Unit *u, KillWho who, KillMode mode, int signo, DBusError } if ((q = cgroup_bonding_kill_list(s->meta.cgroup_bondings, signo, false, pid_set)) < 0) - if (r != -EAGAIN && r != -ESRCH && r != -ENOENT) + if (q != -EAGAIN && q != -ESRCH && q != -ENOENT) r = q; } |