chiark / gitweb /
core: fix return value in error path after sd_event_add_io() failure
authorMichal Schmidt <mschmidt@redhat.com>
Fri, 28 Nov 2014 18:20:59 +0000 (19:20 +0100)
committerMichal Schmidt <mschmidt@redhat.com>
Fri, 28 Nov 2014 18:20:59 +0000 (19:20 +0100)
sd_event_add_io() does not set errno, it returns negative errno.

Noticed during log_*_errno conversions.

src/core/manager.c

index 7b6172896d577c322489d808c1940e212dbe91e0..a84272c905651995da968b493fef316130dede5b 100644 (file)
@@ -692,10 +692,8 @@ static int manager_setup_notify(Manager *m) {
 
         if (!m->notify_event_source) {
                 r = sd_event_add_io(m->event, &m->notify_event_source, m->notify_fd, EPOLLIN, manager_dispatch_notify_fd, m);
-                if (r < 0) {
-                        log_error_errno(r, "Failed to allocate notify event source: %m");
-                        return -errno;
-                }
+                if (r < 0)
+                        return log_error_errno(r, "Failed to allocate notify event source: %m");
 
                 /* Process signals a bit earlier than SIGCHLD, so that we can
                  * still identify to which service an exit message belongs */