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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/dmime/audiopath.c:
> -
>            if (FAILED(hr))
> -
>                goto done;
> -
>        }
> -
>        else if (chunk.type == DMUS_FOURCC_UNFO_LIST)
> -
>            TRACE("Unfo list in PortConfig is ignored\n");
> -
>        else
> -
>            WARN("Unknown %s\n", debugstr_chunk(&chunk));
> -
>        break;
> -
>    default:
> -
>        WARN("Unknown %s\n", debugstr_chunk(&chunk));
> -
>    }
> - }
> -
>
> +done:
> - if (FAILED(hr)) {
> -
>    free(port_config);

Freeing is not enough, you might leak partially parsed data and pchannel_to_buffers entries. Elsewhere I added `<object>_destroy` helpers that I used when needed to delete their entries properly and which can be used in `path_config_IUnknown_Release` too.