Re: [PATCH] avinfo: Fix parsing capabilities

[Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx>]

Hi Pali,
On Sat, Jan 26, 2019 at 12:25 PM Pali Rohár <pali.rohar@xxxxxxxxx> wrote:
>
> Function print_caps() expects capabilities buffer without AVDTP header.
> Previously avinfo somehow worked, because AVDTP header looks like
> capability header with unknown category which was skipped.
> ---
>  tools/avinfo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/avinfo.c b/tools/avinfo.c
> index 02fc1f233..a3298e8c8 100644
> --- a/tools/avinfo.c
> +++ b/tools/avinfo.c
> @@ -809,7 +809,7 @@ static ssize_t avdtp_get_caps(int sk, int seid)
>                 return -1;
>         }
>
> -       print_caps(caps, ret);
> +       print_caps(caps->caps, ret - sizeof(struct getcap_resp));
>
>         return 0;
>  }
> --
> 2.11.0

Applied, thanks.

-- 
Luiz Augusto von Dentz