Bartosz Golaszewski 84700a6d5f 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:01:21 +02:00

..

client

build: replace autotools with meson & ninja

2026-05-04 09:51:03 +02:00

data

build: replace autotools with meson & ninja

2026-05-04 09:51:03 +02:00

lib

build: fix linking against libgpiodbus

2026-05-12 12:32:33 +02:00

manager

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

2026-06-08 10:01:21 +02:00

tests

build: replace autotools with meson & ninja

2026-05-04 09:51:03 +02:00

meson.build

build: replace autotools with meson & ninja

2026-05-04 09:51:03 +02:00