mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-27 04:01:08 +00:00
Code Issues 1 Projects Releases Wiki Activity

decodebin2: Avoid deactivation races

Browse source 
Deactivating pads from two threads isn't 100% MT-safe. There is a
slim chance that the GstPadActivateFunc might be called twice with
the same values (in this case from the cleanup thread *and* from
the GstElement change_state function when going from PAUSED to READY).

In order to avoid that, call any existing cleanup function *before*
calling the parent change_state implementation on downwards state
changes.
This commit is contained in:
Edward Hervey 2017-11-16 06:39:41 +01:00 committed by Edward Hervey
parent 4c2f91d69e
commit e72aa501b0
1 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
gst/playback/gstdecodebin2.c
View file

@ -5372,6 +5372,17 @@ gst_decode_bin_change_state (GstElement * element, GstStateChange transition)
dbin->shutdown = TRUE;
unblock_pads (dbin);
DYN_UNLOCK (dbin);
/* Make sure we don't have cleanup races where
* we might be trying to deactivate pads (in the cleanup thread)
* at the same time as the default element deactivation
* (in PAUSED=>READY) */
g_mutex_lock (&dbin->cleanup_lock);
if (dbin->cleanup_thread) {
g_thread_join (dbin->cleanup_thread);
dbin->cleanup_thread = NULL;
}
g_mutex_unlock (&dbin->cleanup_lock);
default:
break;
}
@ -5405,12 +5416,6 @@ gst_decode_bin_change_state (GstElement * element, GstStateChange transition)
dbin->buffering_status = NULL;
break;
case GST_STATE_CHANGE_READY_TO_NULL:
g_mutex_lock (&dbin->cleanup_lock);
if (dbin->cleanup_thread) {
g_thread_join (dbin->cleanup_thread);
dbin->cleanup_thread = NULL;
}
g_mutex_unlock (&dbin->cleanup_lock);
default:
break;
}