Hi Petri,
On Thu, Dec 1, 2016 at 10:16 PM, Petri Gynther <pgynther@xxxxxxxxxx> wrote:
> Fix the handling of primary services discovery failure.
> ---
> src/shared/gatt-client.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
> index 4386692..84145e7 100644
> --- a/src/shared/gatt-client.c
> +++ b/src/shared/gatt-client.c
> @@ -1052,11 +1052,14 @@ static void discover_primary_cb(bool success, uint8_t att_ecode,
> "Primary service discovery failed."
> " ATT ECODE: 0x%02x", att_ecode);
> /* Reset error in case of not found */
> - if (BT_ATT_ERROR_ATTRIBUTE_NOT_FOUND) {
> + switch (att_ecode) {
> + case BT_ATT_ERROR_ATTRIBUTE_NOT_FOUND:
> success = true;
> att_ecode = 0;
> + goto secondary;
> + default:
> + goto done;
> }
> - goto secondary;
> }
>
> if (!result || !bt_gatt_iter_init(&iter, result)) {
> --
> 2.8.0.rc3.226.g39d4020
Looks like we had similar idea, although this is probably better as
other errors shall be cause the discovery to be interrupted.
--
Luiz Augusto von Dentz
--
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
