Re: [PATCH 0/3] MR7538: winegstreamer: Implement type handler functions for media sink.

[Rémi Bernon (＠rbernon)]

On Tue Mar 11 09:16:16 2025 +0000, Ziqing Hui wrote:
> > There's a test for that case, and it looks like
>
> `IMFMediaSink_AddStreamSink` may also return `E_INVALIDARG` in that
> case. It's flagged as broken
> I can't find that tests, can you tell me where it is? I do find a
> IMFMediaSink_AddStreamSink test will NULL type which is flagged as
> broken, but it's for VideoRenderer in test_evr(), not for media
> sink(file sink).

Yes you're right, it's an evr test. Well, then maybe a test for a media sink with NULL type would tell whether it can be NULL, which seems weird.

If it's indeed not allowed, the logic should probably be adjusted to reject it on stream creation so we can later assume it's not NULL.