Fix screensaver deadlock if terminating as soon the interface is created - closes...

diff --git a/include/vlc_objects.h b/include/vlc_objects.h
index 348e56d0f759990ca2d2a9663f057d9714402aae..7da11129265d5b9f857b9a0b105f88a03f861a83 100644 (file)
--- a/include/vlc_objects.h
+++ b/include/vlc_objects.h
@@ -153,13 +153,17 @@ VLC_EXPORT( vlc_bool_t, __vlc_object_wait, ( vlc_object_t * ) );
 #define vlc_object_wait( obj ) \
     __vlc_object_wait( VLC_OBJECT( obj ) )
 
+/* NOTE: this function is a *temporary* convenience.
+ * See the vlc_object_alive() documentation for a better alternative.
+ */
 static inline
 vlc_bool_t __vlc_object_lock_and_wait( vlc_object_t *obj )
 {
-    vlc_bool_t b;
+    vlc_bool_t b = VLC_TRUE;
 
     vlc_object_lock( obj );
-    b = obj->b_die ? VLC_TRUE : vlc_object_wait( obj );
+    if( vlc_object_alive( obj ) )
+        b = vlc_object_wait( obj );
     vlc_object_unlock( obj );
     return b;
 }
@@ -187,4 +191,8 @@ VLC_EXPORT( void, __vlc_object_kill, ( vlc_object_t * ) );
 #define vlc_object_kill(a) \
     __vlc_object_kill( VLC_OBJECT(a) )
 
+VLC_EXPORT( vlc_bool_t, __vlc_object_alive, ( vlc_object_t * ) );
+#define vlc_object_alive(a) \
+    __vlc_object_alive( VLC_OBJECT(a) )
+
 int vlc_object_waitpipe( vlc_object_t *obj );

diff --git a/modules/misc/screensaver.c b/modules/misc/screensaver.c
index d70fa2e28640f70a26c5ecdb175507bda6b926dc..c2d76afc9d04b4265aa52d53aab708a5ee5d4ea4 100644 (file)
--- a/modules/misc/screensaver.c
+++ b/modules/misc/screensaver.c
@@ -171,13 +171,13 @@ static void Run( intf_thread_t *p_intf )
     p_intf->p_sys->p_connection = dbus_init( p_intf );
 #endif
 
-    for(;;)
+    while( vlc_object_alive( p_intf ) )
     {
         vlc_object_t *p_vout;
 
         /* Check screensaver every 30 seconds */
         if( vlc_object_timedwait( p_intf, mdate() + 30000000 ) < 0 )
-            break;
+            continue;
 
         p_vout = vlc_object_find( p_intf, VLC_OBJECT_VOUT, FIND_ANYWHERE );
 

diff --git a/src/libvlc.sym b/src/libvlc.sym
index e55a318921956d0d5a2e290e0df2ec8bd129ed59..4e64b6243491c422ee60218c21f88b77961e8d79 100644 (file)
--- a/src/libvlc.sym
+++ b/src/libvlc.sym
@@ -17,6 +17,7 @@ NTPtime64
 __str_format
 path_sanitize
 __vlc_object_kill
+__vlc_object_alive
 vlc_b64_encode
 __net_Connect
 __net_ConnectDgram

diff --git a/src/misc/objects.c b/src/misc/objects.c
index 79a64d203bc75c6ce5f5e25531631491a2c6ca12..695b8e7086986f36d67f4f1b37c81c315e454fac 100644 (file)
--- a/src/misc/objects.c
+++ b/src/misc/objects.c
@@ -552,6 +552,36 @@ int __vlc_object_timedwait( vlc_object_t *obj, mtime_t deadline )
 }
 
 
+/**
+ * Checks whether an object has been "killed".
+ * The object lock must be held.
+ *
+ * Typical code for an object thread could be:
+ *
+   vlc_object_lock (self);
+   ...initialization...
+   while (vlc_object_alive (self))
+   {
+       ...preprocessing...
+
+       if (vlc_object_wait (self))
+           continue;
+
+       ...postprocessing...
+   }
+   ...deinitialization...
+   vlc_object_unlock (self);
+ *
+ *
+ * @return true iff the object has not been killed yet
+ */
+vlc_bool_t __vlc_object_alive( vlc_object_t *obj )
+{
+    vlc_assert_locked( &obj->object_lock );
+    return !obj->b_die;
+}
+
+
 /**
  * Signals an object for which the lock is held.
  */