Bartosz Golaszewski 10d34cb4c9 dbus: manager: move g_tree_remove() out of g_assert() in gpiodbus_id_free() ...

g_assert() is compiled out when G_DISABLE_ASSERT is defined, which would
cause the g_tree_remove() call to disappear entirely, leading to ID leaks
and eventual resource exhaustion. Move the side-effecting call outside
the assertion, consistent with the pattern used elsewhere in the daemon.

Fixes: a5ab76da1e ("dbus: add the D-Bus daemon, command-line client and tests")
Assisted-by: Claude Opus 4.6
Reviewed-by: Vincent Fazio <vfazio@xes-inc.com>
Tested-by: Vincent Fazio <vfazio@xes-inc.com>
Link: https://patch.msgid.link/20260604-pre-2-3-fixes-v2-15-781bcaf47475@oss.qualcomm.com
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>

2026-06-08 10:19:17 +02:00

..

client

dbus: client: tweak help text

2025-02-17 09:44:45 +01:00

data

dbus: data: fix SPDX license identifier in D-Bus config file

2024-10-15 20:37:13 +02:00

lib

dbus: don't install generated headers

2025-10-01 15:59:42 +02:00

manager

dbus: manager: move g_tree_remove() out of g_assert() in gpiodbus_id_free()

2026-06-08 10:19:17 +02:00

tests

dbus: add the D-Bus daemon, command-line client and tests

2024-08-13 10:45:25 +02:00

Makefile.am

dbus: add the D-Bus daemon, command-line client and tests

2024-08-13 10:45:25 +02:00