chiark / gitweb /
bus-proxy: no need to negate error codes, log_error_errno() already does it
[elogind.git] / src / bus-proxyd / proxy.c
index 0b8c448757f8f8013d96fe1c2d0b45a713a2b919..1c72eac36691e17a45f1f1d4ffcc8fdd9e0eb029 100644 (file)
@@ -431,7 +431,7 @@ static int process_policy_unlocked(sd_bus *from, sd_bus *to, sd_bus_message *m,
                         return 0;
 
                 /* The message came from the kernel, and is sent to our legacy client. */
-                sd_bus_creds_get_well_known_names(&m->creds, &sender_names);
+                (void) sd_bus_creds_get_well_known_names(&m->creds, &sender_names);
 
                 (void) sd_bus_creds_get_euid(&m->creds, &sender_uid);
                 (void) sd_bus_creds_get_egid(&m->creds, &sender_gid);
@@ -492,7 +492,7 @@ static int process_policy_unlocked(sd_bus *from, sd_bus *to, sd_bus_message *m,
                         if (r < 0)
                                 return handle_policy_error(m, r);
 
-                        sd_bus_creds_get_well_known_names(destination_creds, &destination_names);
+                        (void) sd_bus_creds_get_well_known_names(destination_creds, &destination_names);
 
                         (void) sd_bus_creds_get_euid(destination_creds, &destination_uid);
                         (void) sd_bus_creds_get_egid(destination_creds, &destination_gid);
@@ -576,11 +576,11 @@ static int process_hello(Proxy *p, sd_bus_message *m) {
                 if (p->got_hello)
                         return 0;
 
-                return log_error_errno(-EIO, "First packet isn't hello (it's %s.%s), aborting.", m->interface, m->member);
+                return log_error_errno(EIO, "First packet isn't hello (it's %s.%s), aborting.", m->interface, m->member);
         }
 
         if (p->got_hello)
-                return log_error_errno(-EIO, "Got duplicate hello, aborting.");
+                return log_error_errno(EIO, "Got duplicate hello, aborting.");
 
         p->got_hello = true;