diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-01-08 23:09:37 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-01-08 23:26:07 +0100 |
commit | 690f02f4b8f71b6b1ac84988ebbaf0bd4073c744 (patch) | |
tree | 4e0dd513146ba814f911a0c433b047b053000e97 /src/timesync | |
parent | 1b2a7d92afabd339afbd106e12d6c9ab5a6a00df (diff) | |
download | systemd-690f02f4b8f71b6b1ac84988ebbaf0bd4073c744.tar.gz |
timesyncd,resolved,machinectl: drop calls to sd_event_get_exit_code()
In all three cases, sd_event_loop() will return the exit code anyway.
If sd_event_loop() returns negative, failure is logged and results in an
immediate return. Otherwise, we don't care if sd_event_loop() returns 0
or positive, because the return value feeds into DEFINE_MAIN_FUNCTION(), which
doesn't make the distinction.
Diffstat (limited to 'src/timesync')
-rw-r--r-- | src/timesync/timesyncd.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/timesync/timesyncd.c b/src/timesync/timesyncd.c index 70774d757b..b4f70c0007 100644 --- a/src/timesync/timesyncd.c +++ b/src/timesync/timesyncd.c @@ -169,9 +169,7 @@ static int run(int argc, char *argv[]) { log_debug_errno(r, "Failed to touch %s, ignoring: %m", CLOCK_FILE); } - (void) sd_event_get_exit_code(m->event, &r); - - return r; + return 0; } DEFINE_MAIN_FUNCTION(run); |