embedded snapshot: properly lock/unlock the cache object

diff --git a/src/control/mediacontrol_audio_video.c b/src/control/mediacontrol_audio_video.c
index cf6f0268c96e3708e587f21aa5c98da3960556aa..595532e0f51c1b06d0f96e1dc71c28f6589d11f4 100644 (file)
--- a/src/control/mediacontrol_audio_video.c
+++ b/src/control/mediacontrol_audio_video.c
@@ -94,7 +94,6 @@ mediacontrol_snapshot( mediacontrol_Instance *self,
     vlc_object_release( p_vout );
 
     p_snapshot = ( snapshot_t* ) p_cache->p_private;
-    vlc_object_unlock( p_cache );
     vlc_object_release( p_cache );
     vlc_object_release( p_input );
 

diff --git a/src/video_output/vout_intf.c b/src/video_output/vout_intf.c
index f12b24b5d555f537b1bae06fea9f6aafc7f54d83..5bc53169cdde5429bb4aa711e9057c77f981cead 100644 (file)
--- a/src/video_output/vout_intf.c
+++ b/src/video_output/vout_intf.c
@@ -639,8 +639,7 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic )
         snapshot_t *p_snapshot;
         size_t i_size;
 
-        /* Object must be locked. We will unlock it once we get the
-           snapshot and written it to p_private */
+       vlc_object_lock( p_dest );
         p_dest->p_private = NULL;
 
         /* Save the snapshot to a memory zone */
@@ -650,6 +649,7 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic )
             msg_Err( p_vout, "Could not get snapshot" );
             image_HandlerDelete( p_image );
             vlc_object_signal_unlocked( p_dest );
+           vlc_object_unlock( p_dest );
             vlc_object_release( p_dest );
             return VLC_EGENERIC;
         }
@@ -662,6 +662,7 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic )
             block_Release( p_block );
             image_HandlerDelete( p_image );
             vlc_object_signal_unlocked( p_dest );
+           vlc_object_unlock( p_dest );
             vlc_object_release( p_dest );
             return VLC_ENOMEM;
         }
@@ -679,6 +680,7 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic )
             free( p_snapshot );
             image_HandlerDelete( p_image );
             vlc_object_signal_unlocked( p_dest );
+           vlc_object_unlock( p_dest );
             vlc_object_release( p_dest );
             return VLC_ENOMEM;
         }
@@ -690,6 +692,7 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic )
 
         /* Unlock the object */
         vlc_object_signal_unlocked( p_dest );
+       vlc_object_unlock( p_dest );
         vlc_object_release( p_dest );
 
         image_HandlerDelete( p_image );