multiqueue: never unref queries we do not own

The `query` argument of gst_pad_query is "transfer none".

Query objects are "borrowed" by the pad query handlers and those
should never unref them.

This was leading to double freed queries in a very racy way with nested
GESTimelines.
This commit is contained in:
Thibault Saunier 2019-06-19 23:29:24 -04:00
parent 0257c7813b
commit eaa39398cc

View file

@ -2049,7 +2049,7 @@ next:
out_flushing:
{
if (object)
if (object && !GST_IS_QUERY (object))
gst_mini_object_unref (object);
GST_MULTI_QUEUE_MUTEX_LOCK (mq);