Hi Mikel, On Mon, May 6, 2013 at 11:43 AM, Mikel Astiz <mikel.astiz.oss@xxxxxxxxx> wrote: > +static gboolean service_get_state(const GDBusPropertyTable *property, > + DBusMessageIter *iter, void *data) > +{ > + struct btd_service *service = data; > + const char *str; > + > + str = state2dbus(service->state); > + > + dbus_message_iter_append_basic(iter, DBUS_TYPE_STRING, &str); The code about would break if state2dbus returns NULL for BTD_SERVICE_STATE_UNAVAILABLE, perhaps if service is unavailable it should not have an object path representing it? -- 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