Zbigniew Jędrzejewski-Szmek 62fe94
From 9169b9a8cc43046f92a4c5c85b0b3fe9ca0fb429 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Thomas Hindoe Paaboel Andersen <phomes@gmail.com>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Mon, 15 Sep 2014 22:55:36 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] terminal: sd_bus_error_get_errno returns positive errno
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
The 3 calls to sd_bus_error_get_errno appear to expect a negative
Zbigniew Jędrzejewski-Szmek 62fe94
return value.
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
This patch negates the returned value so it matches the other error
Zbigniew Jędrzejewski-Szmek 62fe94
cases in the 3 functions where sd_bus_error_get_errno is used.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/libsystemd-terminal/sysview.c | 6 +++---
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 3 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/libsystemd-terminal/sysview.c b/src/libsystemd-terminal/sysview.c
Zbigniew Jędrzejewski-Szmek 62fe94
index 2083f5a7e0..fde87d1117 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/libsystemd-terminal/sysview.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/libsystemd-terminal/sysview.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -263,7 +263,7 @@ static int session_take_control_fn(sd_bus *bus,
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
                 log_debug("sysview: %s: TakeControl failed: %s: %s",
Zbigniew Jędrzejewski-Szmek 62fe94
                           session->name, e->name, e->message);
Zbigniew Jędrzejewski-Szmek 62fe94
-                error = sd_bus_error_get_errno(e);
Zbigniew Jędrzejewski-Szmek 62fe94
+                error = -sd_bus_error_get_errno(e);
Zbigniew Jędrzejewski-Szmek 62fe94
         } else {
Zbigniew Jędrzejewski-Szmek 62fe94
                 session->has_control = true;
Zbigniew Jędrzejewski-Szmek 62fe94
                 error = 0;
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1195,7 +1195,7 @@ static int context_ld_list_seats_fn(sd_bus *bus,
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
                 log_debug("sysview: ListSeats on logind failed: %s: %s",
Zbigniew Jędrzejewski-Szmek 62fe94
                           error->name, error->message);
Zbigniew Jędrzejewski-Szmek 62fe94
-                return sd_bus_error_get_errno(error);
Zbigniew Jędrzejewski-Szmek 62fe94
+                return -sd_bus_error_get_errno(error);
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         r = sd_bus_message_enter_container(reply, 'a', "(so)");
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1247,7 +1247,7 @@ static int context_ld_list_sessions_fn(sd_bus *bus,
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
                 log_debug("sysview: ListSessions on logind failed: %s: %s",
Zbigniew Jędrzejewski-Szmek 62fe94
                           error->name, error->message);
Zbigniew Jędrzejewski-Szmek 62fe94
-                return sd_bus_error_get_errno(error);
Zbigniew Jędrzejewski-Szmek 62fe94
+                return -sd_bus_error_get_errno(error);
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         r = sd_bus_message_enter_container(reply, 'a', "(susso)");