diff options
| author | Henrique Dante de Almeida <hdante@profusion.mobi> | 2013-05-27 14:15:34 -0300 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2013-06-19 08:11:35 +0200 |
| commit | 2e2b588a46127a792f779798fac4ae2f2baf2c54 (patch) | |
| tree | e1026ef59ab1a87fdf63fc0e33991cc746595568 /gdbus | |
| parent | 53a0fa8c7cd5c108613a09004c906c2cc7420b86 (diff) | |
| download | ofono-2e2b588a46127a792f779798fac4ae2f2baf2c54.tar.bz2 | |
gdbus: Remove proxies when client disconnects
Remove proxies and generate proxy_removed callbacks even when there are
no corresponding InterfaceRemoved signals.
This patch fixes having zombie gdbus proxy object when a server
disconnects without sending InterfaceRemoved signals. These objects
may interact with new server instances, for example, making
InterfaceAdded signals of new objects with the same name be filtered
out as duplicated, or staying allocated, but unused, if the new server
doesn't reuse the object paths. Note that as a side-effect, the
lifetime of a gdbus proxy becomes stricter: it lives at most for the
duration of a single connection to a single instance of a server
process.
Diffstat (limited to 'gdbus')
| -rw-r--r-- | gdbus/client.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/gdbus/client.c b/gdbus/client.c index f700b7ee..d80e2523 100644 --- a/gdbus/client.c +++ b/gdbus/client.c @@ -1153,6 +1153,10 @@ static DBusHandlerResult message_filter(DBusConnection *connection, if (*new == '\0' && client->unique_name != NULL && g_str_equal(old, client->unique_name) == TRUE) { + + g_list_free_full(client->proxy_list, proxy_free); + client->proxy_list = NULL; + if (client->disconn_func) client->disconn_func(client->dbus_conn, client->disconn_data); |