multiqueue: Fix access to NULL pointer
authorEdward Hervey <edward@centricular.com>
Thu, 20 Jul 2017 12:17:48 +0000 (14:17 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Thu, 20 Jul 2017 12:18:51 +0000 (14:18 +0200)
sq can be NULL.

Also fix commit message (it's the queue we are iterating over that we
are logging, not the one passed as argument).

CID #1415569

plugins/elements/gstmultiqueue.c

index b2d7d44..64cc07b 100644 (file)
@@ -1301,7 +1301,7 @@ calculate_interleave (GstMultiQueue * mq, GstSingleQueue * sq)
     }
     GST_LOG_OBJECT (mq,
         "queue %d , sinktime:%" GST_STIME_FORMAT " low:%" GST_STIME_FORMAT
-        " high:%" GST_STIME_FORMAT, sq->id,
+        " high:%" GST_STIME_FORMAT, oq->id,
         GST_STIME_ARGS (oq->cached_sinktime), GST_STIME_ARGS (low),
         GST_STIME_ARGS (high));
   }
@@ -1310,7 +1310,8 @@ calculate_interleave (GstMultiQueue * mq, GstSingleQueue * sq)
     interleave = high - low;
     /* Padding of interleave and minimum value */
     interleave = (150 * interleave / 100) + mq->min_interleave_time;
-    sq->interleave = interleave;
+    if (sq)
+      sq->interleave = interleave;
 
     interleave = MAX (interleave, other_interleave);