Return-Path: From: Arman Uguray To: linux-bluetooth@vger.kernel.org Cc: Arman Uguray Subject: [PATCH BlueZ v2 4/5] core: device: Don't check ready in service_removed Date: Fri, 30 Jan 2015 16:29:08 -0800 Message-Id: <1422664149-16322-5-git-send-email-armansito@chromium.org> In-Reply-To: <1422664149-16322-1-git-send-email-armansito@chromium.org> References: <1422664149-16322-1-git-send-email-armansito@chromium.org> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: shared/gatt-client clears a given gatt-db if there's an error during its init sequence, even if the given gatt-db was previously populated (e.g. from a cache). This is to make sure that the database contents are at no point invalid. This patch removes a check for bt_gatt_client_is_ready and the corresponding early-return from btd_device's service_removed handler, so that other layers can be notified of invalidated gatt_db_attribute pointers. Change-Id: I8e188d0d0f81398b89f63183ed0708830b6dd9c5 --- src/device.c | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/src/device.c b/src/device.c index 7c8ae74..29a8f23 100644 --- a/src/device.c +++ b/src/device.c @@ -2927,8 +2927,21 @@ static void gatt_service_removed(struct gatt_db_attribute *attr, bt_uuid_t uuid; char uuid_str[MAX_LEN_UUID_STR]; - if (!bt_gatt_client_is_ready(device->client)) - return; + /* + * NOTE: shared/gatt-client clears the database in case of failure. This + * triggers the service_removed callback for all affected services. + * Hence, this function will be called in the following cases: + * + * 1. When a GATT service gets removed due to "Service Changed". + * + * 2. When a GATT service gets removed when the database get cleared + * upon disconnection with a non-bonded device. + * + * 3. When a GATT service gets removed when the database get cleared + * by shared/gatt-client when its initialization procedure fails, + * e.g. due to an ATT protocol error or an unexpected disconnect. + * In this case the gatt-client will not be ready. + */ gatt_db_attribute_get_service_data(attr, &start, &end, NULL, &uuid); -- 2.2.0.rc0.207.ga3a616c