Hi Marcin,
On Thursday 10 of April 2014 20:40:37 Marcin Kraglak wrote:
> ---
> android/gatt.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/android/gatt.c b/android/gatt.c
> index 857ffe0..b246137 100644
> --- a/android/gatt.c
> +++ b/android/gatt.c
> @@ -1150,12 +1150,14 @@ static void handle_client_disconnect(const void *buf, uint16_t len)
> {
> const struct hal_cmd_gatt_client_disconnect *cmd = buf;
> struct gatt_device *dev;
> + bdaddr_t bdaddr;
> uint8_t status;
> char addr[18];
>
> DBG("");
>
> - ba2str((bdaddr_t *)&cmd->bdaddr, addr);
> + android2bdaddr(cmd->bdaddr, &bdaddr);
> + ba2str(&bdaddr, addr);
>
> dev = find_device_by_conn_id(cmd->conn_id);
> if (!dev) {
>
I've pushed this patch but should we do at least sanity check on passed
device address in this command ie. check if connection id and address match?
--
Best regards,
Szymon Janc
--
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
