From b50abd9f4ebe0ed1981578ae3e55ae2cbe6f57fb Mon Sep 17 00:00:00 2001 From: =?utf8?q?Alicia=20Boya=20Garc=C3=ADa?= Date: Mon, 22 Jul 2019 15:06:20 +0000 Subject: [PATCH] downloadbuffer: Check for flush after seek In gst_download_buffer_wait_for_data(), when a seek is made with perform_seek_to_offset() the `qlock` is released temporarily. Therefore, the flushing condition can be set during this period and should be checked. This was not being checked before, causing occasional deadlocks when GST_DOWNLOAD_BUFFER_WAIT_ADD_CHECK() was called. GST_DOWNLOAD_BUFFER_WAIT_ADD_CHECK() assumes that the caller has already checked that we're not flushing before, since this is done when acquiring the lock; so if we release it temporarily somewhere, we need to check for flush again. Without that check, the function would keep waiting for the condition variable to be notified before checking for flushing condition again, and that may very well never happen. This was reproduced when during pad deactivation when running WebKit in gdb. --- plugins/elements/gstdownloadbuffer.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/plugins/elements/gstdownloadbuffer.c b/plugins/elements/gstdownloadbuffer.c index 2fc5954..5e898f9 100644 --- a/plugins/elements/gstdownloadbuffer.c +++ b/plugins/elements/gstdownloadbuffer.c @@ -700,9 +700,15 @@ gst_download_buffer_wait_for_data (GstDownloadBuffer * dlbuf, guint64 offset, } } - if (dlbuf->write_pos != offset) + if (dlbuf->write_pos != offset) { perform_seek_to_offset (dlbuf, offset); + /* perform_seek_to_offset() releases the lock, so we may have been flushed + * during the call. */ + if (dlbuf->srcresult == GST_FLOW_FLUSHING) + goto out_flushing; + } + dlbuf->filling = TRUE; if (dlbuf->write_pos > dlbuf->read_pos) update_levels (dlbuf, dlbuf->write_pos - dlbuf->read_pos); -- 2.7.4