diff options
| author | Szymon Janc <szymon.janc@tieto.com> | 2015-02-23 19:33:00 +0100 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2015-02-24 18:25:17 +0100 |
| commit | a29d429ed3b790de3e730155df67a22d10880f19 (patch) | |
| tree | 5ee854ebbbea9ea47f961f204cd482cf5bdde677 | |
| parent | 6d572ea82b87c981f00f10374df352d38a23724b (diff) | |
| download | pacrunner-a29d429ed3b790de3e730155df67a22d10880f19.tar.gz | |
gdbus: Fix not calling disconnect function
If daemon gets disconnected from D-Bus sender is NULL. Watches that
was explicitly added with NULL sender (ie disconnected_signal in
g_dbus_set_disconnect_function) should be called anyway.
| -rw-r--r-- | gdbus/watch.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/gdbus/watch.c b/gdbus/watch.c index 0d0054c..b60f650 100644 --- a/gdbus/watch.c +++ b/gdbus/watch.c @@ -523,9 +523,7 @@ static DBusHandlerResult message_filter(DBusConnection *connection, member = dbus_message_get_member(message); dbus_message_get_args(message, NULL, DBUS_TYPE_STRING, &arg, DBUS_TYPE_INVALID); - /* Sender is always the owner */ - if (sender == NULL) - return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; + /* If sender != NULL it is always the owner */ for (current = listeners; current != NULL; current = current->next) { data = current->data; @@ -533,6 +531,9 @@ static DBusHandlerResult message_filter(DBusConnection *connection, if (connection != data->connection) continue; + if (!sender && data->owner) + continue; + if (data->owner && g_str_equal(sender, data->owner) == FALSE) continue; |