It's possible that a state object pointer not in the topology node collection gets passed to
session_get_node_object(). Instead of returning the last node when the object is not found, we
should return a NULL so that the state of the last node is not changed by mistake.
--
v6: mf/tests: Test IMFMediaSession::Start().
mf/tests: Add a create_media_session() helper.
mf: Add seeking support for IMFMediaSession::Start().
mf: Add a session_flush_nodes() helper.
mf: Make session_get_node_object() more robust.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3572
On Wed Sep 13 02:57:29 2023 +0000, Alex Henrie wrote:
> You're right that it's unnecessary to cast the void pointer returned
> from malloc or realloc, so I've removed those casts. Thanks for pointing
> that out. As far as the casts in the calls to free, they are all to
> avoid warnings about freeing pointers to const strings, so they can't be removed.
No, never mind, it's necessary to cast the pointers returned from malloc and realloc here because we add 1 to the pointer before assigning it to a variable. It's unusual but it works. Still, we could simplify this code a bit by only calling realloc.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3736#note_45138
These functions are used by Visual Studio 2019, when opening or converting some projects.
Signed-off-by: Lorenzo Ferrillo <lorenzofersteam(a)live.it>
--
v9: kernelbase: Check for PERF_SIZE_LARGE in PerfSetULongLongCounterValue and PerfSetULongCounterValue
https://gitlab.winehq.org/wine/wine/-/merge_requests/3799