summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>2015-02-24 16:06:08 +0000
committerVincent Penquerc'h <vincent.penquerch@collabora.co.uk>2015-02-24 16:07:26 +0000
commit561ddabd97b52229d65ac05a5e281f031e42e621 (patch)
tree1aa3e578ff6f96b22f6250d2dbaf93ee8a04324c
parent0b0c667e57675e6c61d25b268f5f4057c92b10bf (diff)
decodebin: fix deadlock when resetting buffering
This function is static, and only ever called with the expose lock taken. It thus has no reason to take this lock itself. This was introduced by one of my locking fixes from 741355. https://bugzilla.gnome.org/show_bug.cgi?id=741355
-rw-r--r--gst/playback/gstdecodebin2.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/gst/playback/gstdecodebin2.c b/gst/playback/gstdecodebin2.c
index c4fa8572d..6e6b1becb 100644
--- a/gst/playback/gstdecodebin2.c
+++ b/gst/playback/gstdecodebin2.c
@@ -3062,13 +3062,11 @@ gst_decode_bin_reset_buffering (GstDecodeBin * dbin)
return;
GST_DEBUG_OBJECT (dbin, "Reseting multiqueues buffering");
- EXPOSE_LOCK (dbin);
if (dbin->decode_chain) {
CHAIN_MUTEX_LOCK (dbin->decode_chain);
gst_decode_chain_reset_buffering (dbin->decode_chain);
CHAIN_MUTEX_UNLOCK (dbin->decode_chain);
}
- EXPOSE_UNLOCK (dbin);
}
/****