From ac70239020d3ea58a140784a2f7e95d511f7ba13 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Tim-Philipp=20M=C3=BCller?= Date: Sun, 24 Oct 2010 18:50:30 +0100 Subject: [PATCH] qtdemux: fix deadlock in error code path GST_ELEMENT_ERROR must not be called with the object lock held, since it will call gst_object_get_parent() internally, which takes the object lock as well. --- gst/qtdemux/qtdemux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gst/qtdemux/qtdemux.c b/gst/qtdemux/qtdemux.c index ced1e4d..c3ceaa9 100644 --- a/gst/qtdemux/qtdemux.c +++ b/gst/qtdemux/qtdemux.c @@ -4808,9 +4808,9 @@ out_of_samples: } corrupt_file: { + GST_OBJECT_UNLOCK (qtdemux); GST_ELEMENT_ERROR (qtdemux, STREAM, DEMUX, (_("This file is corrupt and cannot be played.")), (NULL)); - GST_OBJECT_UNLOCK (qtdemux); return FALSE; } } -- 2.7.4