[PATCH 2/4] winegstreamer: Don't grab filter_cs in activate_push().

Zebediah Figura z.figura12 at gmail.com
Thu Jan 28 18:40:14 CST 2021


There's no good reason to do this; we're not protecting anything that isn't
already protected by GStreamer locks.

Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
 dlls/winegstreamer/gstdemux.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c
index b2a20934837..e7178bb4dfc 100644
--- a/dlls/winegstreamer/gstdemux.c
+++ b/dlls/winegstreamer/gstdemux.c
@@ -1421,7 +1421,6 @@ static gboolean activate_push(GstPad *pad, gboolean activate)
 {
     struct parser *This = gst_pad_get_element_private(pad);
 
-    EnterCriticalSection(&This->filter.filter_cs);
     if (!activate) {
         TRACE("Deactivating\n");
         if (This->push_thread) {
@@ -1441,7 +1440,6 @@ static gboolean activate_push(GstPad *pad, gboolean activate)
             return FALSE;
         }
     }
-    LeaveCriticalSection(&This->filter.filter_cs);
     return TRUE;
 }
 
-- 
2.30.0




More information about the wine-devel mailing list