Re: [PATCH BlueZ] core/gatt-client: Fix WriteValue always failing after first request

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Fri, Sep 18, 2015 at 4:49 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> WriteValue may use write without response proceduce in case it is
> supported but since that don't have any response write_id is left set
> after first request preventing blocking any new request.
> ---
>  src/gatt-client.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gatt-client.c b/src/gatt-client.c
> index 399133a..6c10a82 100644
> --- a/src/gatt-client.c
> +++ b/src/gatt-client.c
> @@ -946,11 +946,11 @@ static DBusMessage *characteristic_write_value(DBusConnection *conn,
>                 goto fail;
>
>         supported = true;
> -       chrc->write_id = bt_gatt_client_write_without_response(gatt,
> +
> +       if (bt_gatt_client_write_without_response(gatt,
>                                         chrc->value_handle,
>                                         chrc->props & BT_GATT_CHRC_PROP_AUTH,
> -                                       value, value_len);
> -       if (chrc->write_id)
> +                                       value, value_len))
>                 return dbus_message_new_method_return(msg);
>
>  fail:
> --
> 2.4.3

Applied.


-- 
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



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux