access:dshow: fix crash when opening dshow:// fails
authorSteve Lhomme <robUx4@videolabs.io>
Mon, 27 Nov 2017 12:50:10 +0000 (13:50 +0100)
committerJean-Baptiste Kempf <jb@videolan.org>
Mon, 27 Nov 2017 20:46:14 +0000 (21:46 +0100)
The MTA is never acquired but it's released on CommonClose(). So we acquire it
early.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
modules/access/dshow/dshow.cpp

index 2453a9f..f22571e 100644 (file)
@@ -659,12 +659,6 @@ static int CommonOpen( vlc_object_t *p_this, access_sys_t *p_sys,
 
     if( p_sys->pp_streams.empty() ) return VLC_EGENERIC;
 
-    if( vlc_mta_acquire( p_this ) == false )
-    {
-        msg_Err( p_this, "Failed to acquire MTA" );
-        return VLC_EGENERIC;
-    }
-
     return VLC_SUCCESS;
 }
 
@@ -683,6 +677,12 @@ static int DemuxOpen( vlc_object_t *p_this )
 
     ComContext ctx( COINIT_MULTITHREADED );
 
+    if( vlc_mta_acquire( p_this ) == false )
+    {
+        msg_Err( p_this, "Failed to acquire MTA" );
+        return VLC_EGENERIC;
+    }
+
     if( CommonOpen( p_this, p_sys, true ) != VLC_SUCCESS )
     {
         CommonClose( p_this, p_sys );
@@ -790,6 +790,12 @@ static int AccessOpen( vlc_object_t *p_this )
 
     ComContext ctx( COINIT_MULTITHREADED );
 
+    if( vlc_mta_acquire( p_this ) == false )
+    {
+        msg_Err( p_this, "Failed to acquire MTA" );
+        return VLC_EGENERIC;
+    }
+
     if( CommonOpen( p_this, p_sys, false ) != VLC_SUCCESS )
     {
         CommonClose( p_this, p_sys );