diff options
author | Franck Bui <fbui@suse.com> | 2019-03-18 11:48:34 +0100 |
---|---|---|
committer | The Plumber <50238977+systemd-rhel-bot@users.noreply.github.com> | 2019-10-21 16:37:15 +0200 |
commit | f057aa6bb604845fa10ad569bca306e5e1e8fe0d (patch) | |
tree | b06a12678826cac5cbdffdee761f76b6268d13f1 /src/core/service.c | |
parent | 1bf923686a6842f222b1ef5f5174511340c75685 (diff) | |
download | systemd-f057aa6bb604845fa10ad569bca306e5e1e8fe0d.tar.gz |
process-util: introduce pid_is_my_child() helper
No functional changes.
Thanks Renaud Métrich for backporting this to RHEL.
Resolves: #1744972
Diffstat (limited to 'src/core/service.c')
-rw-r--r-- | src/core/service.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/core/service.c b/src/core/service.c index 24f167572a..614ba05d89 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -139,8 +139,6 @@ static void service_unwatch_pid_file(Service *s) { } static int service_set_main_pid(Service *s, pid_t pid) { - pid_t ppid; - assert(s); if (pid <= 1) @@ -159,12 +157,10 @@ static int service_set_main_pid(Service *s, pid_t pid) { s->main_pid = pid; s->main_pid_known = true; + s->main_pid_alien = pid_is_my_child(pid) == 0; - if (get_process_ppid(pid, &ppid) >= 0 && ppid != getpid_cached()) { + if (s->main_pid_alien) log_unit_warning(UNIT(s), "Supervising process "PID_FMT" which is not our child. We'll most likely not notice when it exits.", pid); - s->main_pid_alien = true; - } else - s->main_pid_alien = false; return 0; } |