Create hog connection cb functionality tried to operate on stored on list hidhost device. Adding hidhost device to list should be done before operating on it. It caused sending bogus connecting state change. --- android/hidhost.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/android/hidhost.c b/android/hidhost.c index 846dd57..fdbe885 100644 --- a/android/hidhost.c +++ b/android/hidhost.c @@ -846,9 +846,12 @@ static void bt_hid_connect(const void *buf, uint16_t len) ba2str(&dev->dst, addr); DBG("connecting to %s", addr); + devices = g_slist_append(devices, dev); + if (bt_is_device_le(&dst)) { if (!hog_connect(dev)) { status = HAL_STATUS_FAILED; + hid_device_remove(dev); goto failed; } goto done; @@ -864,8 +867,6 @@ static void bt_hid_connect(const void *buf, uint16_t len) } done: - devices = g_slist_append(devices, dev); - if (dev->state == HAL_HIDHOST_STATE_DISCONNECTED) bt_hid_notify_state(dev, HAL_HIDHOST_STATE_CONNECTING); -- 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