Hi Pauli,
On Wed, Feb 1, 2023 at 10:57 AM Pauli Virtanen <pav@xxxxxx> wrote:
>
> The "Vendor" field is optional, and should have an initialized valid
> default value.
It is probably a good idea to include logs where the problem is detected.
> ---
> profiles/audio/media.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/profiles/audio/media.c b/profiles/audio/media.c
> index d96367454..a62755f69 100644
> --- a/profiles/audio/media.c
> +++ b/profiles/audio/media.c
> @@ -2551,7 +2551,7 @@ static void app_register_endpoint(void *data, void *user_data)
> const char *uuid;
> gboolean delay_reporting = FALSE;
> uint8_t codec;
> - struct vendor vendor;
> + struct vendor vendor = { 0 };
I guess {}; or { 0,0 }; are clearer here since there are 2 fields in
the struct, although it is probably safer to go with memset(&vendor,
0, sizeof(vendor)).
> struct bt_bap_pac_qos qos;
> uint8_t *capabilities = NULL;
> int size = 0;
> --
> 2.39.1
>
--
Luiz Augusto von Dentz
