[vlc-devel] [PATCH 5/6] mmal/vout: Remove bad locking

Julian Scheel julian at jusst.de
Fri Aug 29 09:09:37 CEST 2014


This removes some bad locking. In fact this lock could cause a deadlock on
shutdown as the lock is held by the main thread when shutting down and this
function might be called as a result of shutting the ports down.

Signed-off-by: Julian Scheel <julian at jusst.de>
---
 modules/hw/mmal/codec.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/modules/hw/mmal/codec.c b/modules/hw/mmal/codec.c
index 687e25b..d2424ea 100644
--- a/modules/hw/mmal/codec.c
+++ b/modules/hw/mmal/codec.c
@@ -449,10 +449,8 @@ static picture_t *decode(decoder_t *dec, block_t **pblock)
      * Configure output port if necessary
      */
     if (sys->output_format) {
-        vlc_mutex_lock(&sys->mutex);
         if (change_output_format(dec) < 0)
             msg_Err(dec, "Failed to change output port format");
-        vlc_mutex_unlock(&sys->mutex);
     }
 
     /*
-- 
2.1.0




More information about the vlc-devel mailing list