Hi Arik, On Sun, Mar 25, 2012 at 5:38 PM, Arik Nemtsov <arik@xxxxxxxxxx> wrote: > +void imm_alert_unregister(struct btd_adapter *adapter) > +{ > + struct imm_alert_adapter *imadapter; > + > + imadapter = find_imm_alert_adapter(adapter); > + if (!adapter) > + return; The check above looks strange. Maybe you meant "if (!imadapter)"? > + > + g_slist_foreach(imadapter->connected_devices, remove_condev_list_item, > + NULL); > + dbus_connection_unref(imadapter->conn); > + > + imm_alert_adapters = g_slist_remove(imm_alert_adapters, imadapter); > + g_free(imadapter); > +} Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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