Re: [PATCH v2 0/3] MR4479: winegstreamer: Implement wg_muxer_finalize.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/media_sink.c:
>  if (FAILED(hr = async_command_create(op, &command)))
>      return hr;
>
> - return MFPutWorkItem(MFASYNC_CALLBACK_QUEUE_STANDARD, &media_sink->async_callback, &command->IUnknown_iface);
> - if (FAILED(hr = MFPutWorkItem(MFASYNC_CALLBACK_QUEUE_STANDARD,
> -
>        &media_sink->async_callback, &command->IUnknown_iface)))
> -
>    IUnknown_Release(&command->IUnknown_iface);
> -
> - return hr;

Eh sorry, actually it's not only when it fails, it's always leaking it. This should be:

```suggestion:-4+0
    hr = MFPutWorkItem(MFASYNC_CALLBACK_QUEUE_STANDARD,
            &media_sink->async_callback, &command->IUnknown_iface));
    IUnknown_Release(&command->IUnknown_iface);

return hr;
```