summaryrefslogtreecommitdiff
path: root/gdbus
diff options
context:
space:
mode:
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>2014-08-11 17:58:37 +0300
committerJohan Hedberg <johan.hedberg@intel.com>2014-08-12 10:58:32 +0300
commit274860fbc5fe458b4c4087f8c649387f06d68b29 (patch)
tree6e86698fc7ad6d55d55e935e7095ed47771860de /gdbus
parent3d8a888a7596435fbd5da28f23dddfc46ddff0e7 (diff)
downloadbluez-274860fbc5fe458b4c4087f8c649387f06d68b29.tar.gz
Revert "gdbus: Don't include just added interfaces in GetManagedObjects"
This actually creates a problem since the code now does flush any pending message on the queue any signal will be send right away before the actual reply to GetManagedObjects which will not contain those interfaces.
Diffstat (limited to 'gdbus')
-rw-r--r--gdbus/object.c8
1 files changed, 1 insertions, 7 deletions
diff --git a/gdbus/object.c b/gdbus/object.c
index 13cf9a956..4d5a64cb7 100644
--- a/gdbus/object.c
+++ b/gdbus/object.c
@@ -1088,7 +1088,6 @@ static const GDBusMethodTable introspect_methods[] = {
static void append_interfaces(struct generic_data *data, DBusMessageIter *iter)
{
DBusMessageIter array;
- GSList *l;
dbus_message_iter_open_container(iter, DBUS_TYPE_ARRAY,
DBUS_DICT_ENTRY_BEGIN_CHAR_AS_STRING
@@ -1100,12 +1099,7 @@ static void append_interfaces(struct generic_data *data, DBusMessageIter *iter)
DBUS_DICT_ENTRY_END_CHAR_AS_STRING
DBUS_DICT_ENTRY_END_CHAR_AS_STRING, &array);
- for (l = data->interfaces; l != NULL; l = l->next) {
- if (g_slist_find(data->added, l->data))
- continue;
-
- append_interface(l->data, &array);
- }
+ g_slist_foreach(data->interfaces, append_interface, &array);
dbus_message_iter_close_container(iter, &array);
}