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:
> ```suggestion:-1+0
>     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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4852#note_57976