]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
hostnamed,localed,timedate: properly propagate errors from sd_event_add_signal()
authorLennart Poettering <lennart@poettering.net>
Mon, 12 Nov 2018 12:31:03 +0000 (13:31 +0100)
committerLennart Poettering <lennart@poettering.net>
Mon, 12 Nov 2018 16:51:34 +0000 (17:51 +0100)
main() can't doesn't expect negative error codes as return.

src/hostname/hostnamed.c
src/locale/localed.c
src/timedate/timedated.c

index d8074e6f5fb454ffba993e4ec9238e53cef2eb18..a0535070b3ffb48644d37d3ef662e73b95896f81 100644 (file)
@@ -732,12 +732,16 @@ int main(int argc, char *argv[]) {
         (void) sd_event_set_watchdog(event, true);
 
         r = sd_event_add_signal(event, NULL, SIGINT, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGINT handler: %m");
+                goto finish;
+        }
 
         r = sd_event_add_signal(event, NULL, SIGTERM, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGTERM handler: %m");
+                goto finish;
+        }
 
         r = connect_bus(&context, event, &bus);
         if (r < 0)
index 1f0c3ac74fa4a867f3aa552dd6426f213d633c24..c949fa3831d71dfe19fb6af473a61113fe7bfd37 100644 (file)
@@ -751,12 +751,16 @@ int main(int argc, char *argv[]) {
         (void) sd_event_set_watchdog(event, true);
 
         r = sd_event_add_signal(event, NULL, SIGINT, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGINT handler: %m");
+                goto finish;
+        }
 
         r = sd_event_add_signal(event, NULL, SIGTERM, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGTERM handler: %m");
+                goto finish;
+        }
 
         r = connect_bus(&context, event, &bus);
         if (r < 0)
index 8cf5e8d48d02c4bee57ebf3ef8d521e2d91552de..71f2f6f69b4cb2ad4ea07e9b37bd17f490f3b9b0 100644 (file)
@@ -987,12 +987,16 @@ int main(int argc, char *argv[]) {
         (void) sd_event_set_watchdog(event, true);
 
         r = sd_event_add_signal(event, NULL, SIGINT, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGINT handler: %m");
+                goto finish;
+        }
 
         r = sd_event_add_signal(event, NULL, SIGTERM, NULL, NULL);
-        if (r < 0)
-                return r;
+        if (r < 0) {
+                log_error_errno(r, "Failed to install SIGTERM handler: %m");
+                goto finish;
+        }
 
         r = connect_bus(&context, event, &bus);
         if (r < 0)