Hi Luiz, On Thu, Oct 30, 2014 at 6:57 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> > > --- > src/shared/gatt-db.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/src/shared/gatt-db.c b/src/shared/gatt-db.c > index 60a1b23..74cd0be 100644 > --- a/src/shared/gatt-db.c > +++ b/src/shared/gatt-db.c > @@ -841,7 +841,10 @@ uint16_t gatt_db_attribute_get_start_handle(struct gatt_db_attribute *attrib) > > uint16_t gatt_db_attribute_get_end_handle(struct gatt_db_attribute *attrib) > { > - return 0; > + if (!attrib) > + return 0; > + > + return attrib->handle + attrib->service->num_handles - 1; Is this right? If I have a non-first handle (one created by gatt_db_add_characteristic) this will give me a handle off the end of the service. I'm not sure of the meaning of start/end handle here. > } > > const bt_uuid_t *gatt_db_attribute_get_type(struct gatt_db_attribute *attrib) > -- > 1.9.3 > > -- > 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 -- Michael Janssen -- 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