From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html