Zbigniew Jędrzejewski-Szmek db38be
From 547e849b722e597d70ba069e2cb8931317d66e00 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek db38be
From: Lukasz Skalski <l.skalski@samsung.com>
Zbigniew Jędrzejewski-Szmek db38be
Date: Fri, 10 Oct 2014 16:42:19 +0200
Zbigniew Jędrzejewski-Szmek db38be
Subject: [PATCH] bus-proxyd: fix compatibility with old dbus-1
Zbigniew Jędrzejewski-Szmek db38be
Zbigniew Jędrzejewski-Szmek db38be
'ListQueuedOwners' method should return 'NameHasNoOwner' error
Zbigniew Jędrzejewski-Szmek db38be
if chosen name is not available on bus.
Zbigniew Jędrzejewski-Szmek db38be
---
Zbigniew Jędrzejewski-Szmek db38be
 src/bus-proxyd/bus-proxyd.c | 9 +++++++++
Zbigniew Jędrzejewski-Szmek db38be
 1 file changed, 9 insertions(+)
Zbigniew Jędrzejewski-Szmek db38be
Zbigniew Jędrzejewski-Szmek db38be
diff --git a/src/bus-proxyd/bus-proxyd.c b/src/bus-proxyd/bus-proxyd.c
Zbigniew Jędrzejewski-Szmek db38be
index 4f44825679..52498f33d2 100644
Zbigniew Jędrzejewski-Szmek db38be
--- a/src/bus-proxyd/bus-proxyd.c
Zbigniew Jędrzejewski-Szmek db38be
+++ b/src/bus-proxyd/bus-proxyd.c
Zbigniew Jędrzejewski-Szmek db38be
@@ -733,6 +733,7 @@ static int process_driver(sd_bus *a, sd_bus *b, sd_bus_message *m) {
Zbigniew Jędrzejewski-Szmek db38be
                 struct kdbus_cmd_free cmd_free;
Zbigniew Jędrzejewski-Szmek db38be
                 struct kdbus_cmd_name *name;
Zbigniew Jędrzejewski-Szmek db38be
                 _cleanup_strv_free_ char **owners = NULL;
Zbigniew Jędrzejewski-Szmek db38be
+                _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL;
Zbigniew Jędrzejewski-Szmek db38be
                 char *arg0;
Zbigniew Jędrzejewski-Szmek db38be
                 int err = 0;
Zbigniew Jędrzejewski-Szmek db38be
 
Zbigniew Jędrzejewski-Szmek db38be
@@ -743,6 +744,14 @@ static int process_driver(sd_bus *a, sd_bus *b, sd_bus_message *m) {
Zbigniew Jędrzejewski-Szmek db38be
                 if (!service_name_is_valid(arg0))
Zbigniew Jędrzejewski-Szmek db38be
                         return synthetic_reply_method_errno(m, -EINVAL, NULL);
Zbigniew Jędrzejewski-Szmek db38be
 
Zbigniew Jędrzejewski-Szmek db38be
+                r = sd_bus_get_owner(a, arg0, 0, NULL);
Zbigniew Jędrzejewski-Szmek db38be
+                if (r == -ESRCH || r == -ENXIO) {
Zbigniew Jędrzejewski-Szmek db38be
+                        sd_bus_error_setf(&error, SD_BUS_ERROR_NAME_HAS_NO_OWNER, "Could not get owners of name '%s': no such name.", arg0);
Zbigniew Jędrzejewski-Szmek db38be
+                        return synthetic_reply_method_errno(m, r, &error);
Zbigniew Jędrzejewski-Szmek db38be
+                }
Zbigniew Jędrzejewski-Szmek db38be
+                if (r < 0)
Zbigniew Jędrzejewski-Szmek db38be
+                        return synthetic_reply_method_errno(m, r, NULL);
Zbigniew Jędrzejewski-Szmek db38be
+
Zbigniew Jędrzejewski-Szmek db38be
                 cmd.flags = KDBUS_NAME_LIST_QUEUED;
Zbigniew Jędrzejewski-Szmek db38be
                 r = ioctl(a->input_fd, KDBUS_CMD_NAME_LIST, &cmd);
Zbigniew Jędrzejewski-Szmek db38be
                 if (r < 0)