diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-01-14 21:05:17 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-01-14 21:05:17 +0100 |
commit | fbeefb45ac1a257a0c5af975ad26d68ed6c39fda (patch) | |
tree | 19e7a067b637b1c79a83a8c586043cb5cb110cc4 | |
parent | e5e991a11c7749f8cb424e99ca4206264b0cf836 (diff) |
service: for Type=forking services, ignore exit status of main process depending on ExecStart's ignore setting
https://bugzilla.redhat.com/show_bug.cgi?id=860464
-rw-r--r-- | src/core/service.c | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/src/core/service.c b/src/core/service.c index 8e9e11236d..7eaac0dc4c 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -2926,15 +2926,25 @@ static void service_sigchld_event(Unit *u, pid_t pid, int code, int status) { s->main_pid = 0; exec_status_exit(&s->main_exec_status, &s->exec_context, pid, code, status); - /* If this is not a forking service than the main - * process got started and hence we copy the exit - * status so that it is recorded both as main and as - * control process exit status */ if (s->main_command) { + /* If this is not a forking service than the + * main process got started and hence we copy + * the exit status so that it is recorded both + * as main and as control process exit + * status */ + s->main_command->exec_status = s->main_exec_status; if (s->main_command->ignore) f = SERVICE_SUCCESS; + } else if (s->exec_command[SERVICE_EXEC_START]) { + + /* If this is a forked process, then we should + * ignore the return value if this was + * configured for the starter process */ + + if (s->exec_command[SERVICE_EXEC_START]->ignore) + f = SERVICE_SUCCESS; } log_struct(f == SERVICE_SUCCESS ? LOG_DEBUG : LOG_NOTICE, |