mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-01-02 21:48:55 +00:00
Code Issues 1 Projects Releases Wiki Activity

omx: use recursive_lock inside set_last_error(), since this function may be called from an event handler

Browse source
This commit is contained in:
George Kiagiadakis 2012-06-21 20:21:03 +03:00
parent 6a7743b37f
commit 8a669ee308
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
omx/gstomx.c
View file

@ -734,14 +734,14 @@ gst_omx_component_set_last_error (GstOMXComponent * comp, OMX_ERRORTYPE err)
GST_ERROR_OBJECT (comp->parent, "Setting last error: %s (0x%08x)",
gst_omx_error_to_string (err), err);
gst_omx_rec_mutex_lock (&comp->state_lock);
gst_omx_rec_mutex_recursive_lock (&comp->state_lock);
/* We only set the first error ever from which
* we can't recover anymore.
*/
if (comp->last_error == OMX_ErrorNone)
comp->last_error = err;
g_cond_broadcast (comp->state_cond);
gst_omx_rec_mutex_unlock (&comp->state_lock);
gst_omx_rec_mutex_recursive_unlock (&comp->state_lock);
/* Now notify all ports, no locking needed
* here because the ports are allocated in the
@ -752,9 +752,9 @@ gst_omx_component_set_last_error (GstOMXComponent * comp, OMX_ERRORTYPE err)
for (i = 0; i < n; i++) {
GstOMXPort *tmp = g_ptr_array_index (comp->ports, i);
gst_omx_rec_mutex_lock (&tmp->port_lock);
gst_omx_rec_mutex_recursive_lock (&tmp->port_lock);
g_cond_broadcast (tmp->port_cond);
gst_omx_rec_mutex_unlock (&tmp->port_lock);
gst_omx_rec_mutex_recursive_unlock (&tmp->port_lock);
}
}