---

Hi,

While using gdbus on some other code, I found out that bug around 
g_dbus_remove_all_watches() usage.

I guess it should be fixed in all projects using gdbus.

It could be fixed also via removing "g_free(cb);" in the previous for() loop 
instead, but I preferred
not, since it would then re-loop again on data->callbacks in filter_data_free().

Tomasz

 gdbus/watch.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gdbus/watch.c b/gdbus/watch.c
index a402ca9..8d40fa7 100644
--- a/gdbus/watch.c
+++ b/gdbus/watch.c
@@ -309,6 +309,9 @@ static void filter_data_call_and_free(struct filter_data 
*data)
                g_free(cb);
        }
 
+       g_slist_free(data->callbacks);
+       data->callbacks = NULL;
+
        filter_data_free(data);
 }
 
-- 
1.7.12

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to