sd-bus: fix parsing of KDBUS_CMD_LIST

We *must not* assume that an entry returned by KDBUS_CMD_LIST only
carries a single KDBUS_ITEM_OWNED_NAME. Similarly, we already parse
multiple such items for message-metadata, so make sure we support the
same on KDBUS_CMD_LIST.

By relying on the kernel to return all names separately, we limit the
kernel API significantly. Stop this and let the kernel decide how to
return its data.

1 files changed, 17 insertions, 15 deletions

diff --git a/src/bus-proxyd/driver.c b/src/bus-proxyd/driver.c
index 1cb5ea5008..aa94c6f51b 100644
--- a/src/bus-proxyd/driver.c
+++ b/src/bus-proxyd/driver.c
@@ -441,27 +441,29 @@ int bus_proxy_process_driver(Proxy *p, sd_bus *a, sd_bus *b, sd_bus_message *m,
                 name_list = (struct kdbus_info *) ((uint8_t *) a->kdbus_buffer + cmd.offset);
 
                 KDBUS_FOREACH(name, name_list, cmd.list_size) {
-                        const char *entry_name = NULL;
                         struct kdbus_item *item;
                         char *n;
 
-                        KDBUS_ITEM_FOREACH(item, name, items)
-                                if (item->type == KDBUS_ITEM_OWNED_NAME)
-                                        entry_name = item->name.name;
-
-                        if (!streq_ptr(entry_name, arg0))
-                                continue;
-
-                        if (asprintf(&n, ":1.%llu", (unsigned long long) name->id) < 0) {
-                                err  = -ENOMEM;
-                                break;
+                        KDBUS_ITEM_FOREACH(item, name, items) {
+                                if (item->type == KDBUS_ITEM_OWNED_NAME) {
+                                        if (!streq_ptr(item->name.name, arg0))
+                                                continue;
+
+                                        if (asprintf(&n, ":1.%llu", (unsigned long long) name->id) < 0) {
+                                                err  = -ENOMEM;
+                                                break;
+                                        }
+
+                                        r = strv_consume(&owners, n);
+                                        if (r < 0) {
+                                                err = r;
+                                                break;
+                                        }
+                                }
                         }
 
-                        r = strv_consume(&owners, n);
-                        if (r < 0) {
-                                err = r;
+                        if (err < 0)
                                 break;
-                        }
                 }
 
                 r = bus_kernel_cmd_free(a, cmd.offset);