From 09b4f050e04203ece28add79d419910801072ef7 Mon Sep 17 00:00:00 2001 From: Ben Browitt Date: Thu, 20 Aug 2015 14:11:56 +0300 Subject: [PATCH] videoaggregator: Always set the pad's buffer_vinfo when storing a buffer Otherwise it might be unset, and then the buffer is used and gst_video_frame_map() will crash because of invalid video-info. https://bugzilla.gnome.org/show_bug.cgi?id=753805 --- gst-libs/gst/video/gstvideoaggregator.c | 1 + 1 file changed, 1 insertion(+) diff --git a/gst-libs/gst/video/gstvideoaggregator.c b/gst-libs/gst/video/gstvideoaggregator.c index b8d29ee..fe41e02 100644 --- a/gst-libs/gst/video/gstvideoaggregator.c +++ b/gst-libs/gst/video/gstvideoaggregator.c @@ -1025,6 +1025,7 @@ gst_videoaggregator_fill_queues (GstVideoAggregator * vagg, GST_DEBUG_OBJECT (pad, "buffer duration is -1, start_time < " "output_start_time. Discarding old buffer"); gst_buffer_replace (&pad->buffer, buf); + pad->buffer_vinfo = *vinfo; gst_buffer_unref (buf); gst_aggregator_pad_drop_buffer (bpad); need_more_data = TRUE; -- 2.7.4