diff mbox series

[libgpiod,3/3] dbus: client: monitor: free the line list at exit

Message ID 20241007-dbus-memory-fixes-v1-3-0d56d1aa032d@linaro.org
State New
Headers show
Series dbus: client: fix memory leaks and errors | expand

Commit Message

Bartosz Golaszewski Oct. 7, 2024, 7:30 p.m. UTC 
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

Free the list of line proxies stored for the duration of the program at
exit to fix a memory leak.

Note: these objects need to be kept alive for D-Bus events to work.

Fixes: a5ab76da1e0a ("dbus: add the D-Bus daemon, command-line client and tests")
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
 dbus/client/monitor.c | 1 +
 1 file changed, 1 insertion(+)
diff mbox series

Patch

diff --git a/dbus/client/monitor.c b/dbus/client/monitor.c
index 3bd25b8..df7eb51 100644
--- a/dbus/client/monitor.c
+++ b/dbus/client/monitor.c
@@ -185,6 +185,7 @@  int gpiocli_monitor_main(int argc, char **argv)
 
 	g_main_loop_run(loop);
 
+	g_list_free_full(data.lines, g_object_unref);
 	g_bus_unwatch_name(watch_id);
 
 	return EXIT_SUCCESS;