[PATCH] winegstreamer: Create media source read thread after initializing wg_parser.

Paul Gofman pgofman at codeweavers.com
Wed Aug 25 06:43:39 CDT 2021


Fixes race condition as wg_parser is used right away in the created
thread.

Signed-off-by: Paul Gofman <pgofman at codeweavers.com>
---
 dlls/winegstreamer/media_source.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/winegstreamer/media_source.c b/dlls/winegstreamer/media_source.c
index 05544ef7abe..01ab626254a 100644
--- a/dlls/winegstreamer/media_source.c
+++ b/dlls/winegstreamer/media_source.c
@@ -1315,8 +1315,6 @@ static HRESULT media_source_constructor(IMFByteStream *bytestream, struct media_
     if (FAILED(hr = MFAllocateWorkQueue(&object->async_commands_queue)))
         goto fail;
 
-    object->read_thread = CreateThread(NULL, 0, read_thread, object, 0, NULL);
-
     if (!(parser = unix_funcs->wg_decodebin_parser_create()))
     {
         hr = E_OUTOFMEMORY;
@@ -1324,6 +1322,8 @@ static HRESULT media_source_constructor(IMFByteStream *bytestream, struct media_
     }
     object->wg_parser = parser;
 
+    object->read_thread = CreateThread(NULL, 0, read_thread, object, 0, NULL);
+
     object->state = SOURCE_OPENING;
 
     if (FAILED(hr = unix_funcs->wg_parser_connect(parser, file_size)))
-- 
2.31.1




More information about the wine-devel mailing list