Re: [PATCH v10 0/4] MR4852: Add support from AudioPathConfig

[Yuxuan Shui (＠yshui)]

On Thu Jan 18 15:08:25 2024 +0000, Rémi Bernon wrote:
>     if (FAILED(hr))
>         free(pchannel_to_buffer);
>
> Fwiw I would do the same as below, moving the `list_add_tail` call here.
> Maybe even returning an error if hr is successful but pchannel_to_buffer
> is NULL.

No there can be multiple `pchannel_to_buffer`s added to the list here. `pchannel_to_buffer == NULL` is the expected condition when the function returns.