From f3e175bd7884c7c84b68b7167d5e4dc805d374c5 Mon Sep 17 00:00:00 2001 From: "Dirk-Jan C. Binnema" Date: Mon, 29 Jul 2013 11:42:48 -0700 Subject: [PATCH] attrib: Update gatt_service_add size assert for dealing with 128-bit UUIDs To paraphrase Anderson Lizardo: "h" was overflowing for 128-bit services because the handle is incremented after each attribute is added so at the end we had: 0xffff + 1 == 0 (h is uint16_t). As it seems more complex to try to remove the overflow without affecting the logic too much, we update the assertion. --- attrib/gatt-service.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/attrib/gatt-service.c b/attrib/gatt-service.c index 4b02d39e4..1b84c893b 100644 --- a/attrib/gatt-service.c +++ b/attrib/gatt-service.c @@ -341,7 +341,7 @@ gboolean gatt_service_add(struct btd_adapter *adapter, uint16_t uuid, } g_assert(size < USHRT_MAX); - g_assert(h - start_handle == (uint16_t) size); + g_assert(h == 0 || (h - start_handle == (uint16_t) size)); g_slist_free_full(chrs, free_gatt_info); return TRUE; -- 2.47.3