[PATCH BlueZ 2/2 v2] attrib: Don't attempt to unregister event id 0
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: 2013-01-28 21:44:01
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Luiz Augusto von Dentz <redacted> Id 0 is considered invalid so the code should not even try to lookup for it in the event list instead print a warning and return FALSE immediatelly. --- attrib/gattrib.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/attrib/gattrib.c b/attrib/gattrib.c
index 58f19d0..01c19f9 100644
--- a/attrib/gattrib.c
+++ b/attrib/gattrib.c@@ -719,6 +719,11 @@ gboolean g_attrib_unregister(GAttrib *attrib, guint id) struct event *evt; GSList *l; + if (id == 0) { + warn("%s: invalid id", __FUNCTION__); + return FALSE; + } + l = g_slist_find_custom(attrib->events, GUINT_TO_POINTER(id), event_cmp_by_id); if (l == NULL)
--
1.8.1