This patch changes GATT_CHRC_IFACE to GATT_DESC_IFACE. The example do not test this case and this is the reason why nobody detected it yet. Cc: Szymon Janc <szymon.janc@xxxxxxxxxxx> Suggested-by: Florent Revest <revestflo@xxxxxxxxx> Signed-off-by: Alexander Aring <aar@xxxxxxxxxxxxxx> --- Florent Revest reported this via irc and I put my "send-patches" hat on to send this fix to mainline. Review carefully, not tested in any case. test/example-gatt-server | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/example-gatt-server b/test/example-gatt-server index 44ceb70..bac1687 100755 --- a/test/example-gatt-server +++ b/test/example-gatt-server @@ -234,7 +234,7 @@ class Descriptor(dbus.service.Object): if interface != GATT_DESC_IFACE: raise InvalidArgsException() - return self.get_properties()[GATT_CHRC_IFACE] + return self.get_properties()[GATT_DESC_IFACE] @dbus.service.method(GATT_DESC_IFACE, in_signature='a{sv}', -- 2.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