winegstreamer: Do not block forever if EOS races with command queue.

Currently, the winegstreamer media source checks for EOS when RequestSample() is called, but doesn't handle the cases when EOS is detected between the RequestSample() call and the moment when the request is popped from the command queue and serviced. This can result in the media source waiting forever for a sample and get stuck. This commit fixes the bug by adding a check for EOS in wait_for_event(). This commit fixes Medieval Dynasty hanging on developer logos on the Steam Deck. Signed-off-by: Giovanni Mascellani <gmascellani@codeweavers.com> Signed-off-by: Zebediah Figura <zfigura@codeweavers.com> Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winegstreamer: Do not block forever if EOS races with command queue.
4853f65c · Giovanni Mascellani · Alexandre Julliard · 64b2ce4b · 4853f65c
Commit 4853f65c authored Feb 09, 2022 by Giovanni Mascellani Committed by Alexandre Julliard Feb 22, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

media_source.c dlls/winegstreamer/media_source.c +6 -0

No files found.
--- a/dlls/winegstreamer/media_source.c
+++ b/dlls/winegstreamer/media_source.c
@@ -535,6 +535,12 @@ static void wait_on_sample(struct media_stream *stream, IUnknown *token)

    TRACE("%p, %p\n", stream, token);

+    if (stream->eos)
+    {
+        IMFMediaEventQueue_QueueEventParamVar(stream->event_queue, MEError, &GUID_NULL, MF_E_END_OF_STREAM, &empty_var);
+        return;
+    }
+
    for (;;)
    {
        if (!wg_parser_stream_get_event(stream->wg_stream, &event))