Re: [PATCH v4 0/1] MR9742: mf: Prevent invalid topologies from being started.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/session.c:
> hr = IMFTopoLoader_Load(session->topo_loader, topology, &resolved_topology, NULL /* FIXME? */);
>              if (SUCCEEDED(hr))
>                  hr = session_init_media_types(resolved_topology);
> +            else
> +                topology = NULL;
>
> if (SUCCEEDED(hr))
>              {

I think it makes more sense to move this few lines down. I do `if (SUCCEEDED(hr)) topology = resolved_topology; else topology = NULL;`

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9742#note_131844