From d8e681bb6317e1c26ca522e15836d13a6d9b14cb Mon Sep 17 00:00:00 2001 From: Aleksander Wabik Date: Thu, 17 Mar 2016 19:24:32 +0000 Subject: h264parse, h265parse: fix handling of downstream force-key-unit events The parser handles the downstream force-key-unit event incorrectly, it tries to parse it as an upstream force-key-unit event, does not check the return value, and then uses uninitialized memory in "all_headers" boolean variable. https://bugzilla.gnome.org/show_bug.cgi?id=763793 --- gst/videoparsers/gsth264parse.c | 7 +++++-- gst/videoparsers/gsth265parse.c | 7 +++++-- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/gst/videoparsers/gsth264parse.c b/gst/videoparsers/gsth264parse.c index 8907eb41d..28e8f318f 100644 --- a/gst/videoparsers/gsth264parse.c +++ b/gst/videoparsers/gsth264parse.c @@ -2140,8 +2140,11 @@ check_pending_key_unit_event (GstEvent * pending_event, stream_time = gst_segment_to_stream_time (segment, GST_FORMAT_TIME, timestamp); - gst_video_event_parse_upstream_force_key_unit (pending_event, - NULL, &all_headers, &count); + if (!gst_video_event_parse_upstream_force_key_unit (pending_event, + NULL, &all_headers, &count)) { + gst_video_event_parse_downstream_force_key_unit (pending_event, NULL, + NULL, NULL, &all_headers, &count); + } event = gst_video_event_new_downstream_force_key_unit (timestamp, stream_time, diff --git a/gst/videoparsers/gsth265parse.c b/gst/videoparsers/gsth265parse.c index 5a1984d84..5a8f291ee 100644 --- a/gst/videoparsers/gsth265parse.c +++ b/gst/videoparsers/gsth265parse.c @@ -1714,8 +1714,11 @@ check_pending_key_unit_event (GstEvent * pending_event, GstSegment * segment, stream_time = gst_segment_to_stream_time (segment, GST_FORMAT_TIME, timestamp); - gst_video_event_parse_upstream_force_key_unit (pending_event, - NULL, &all_headers, &count); + if (!gst_video_event_parse_upstream_force_key_unit (pending_event, + NULL, &all_headers, &count)) { + gst_video_event_parse_downstream_force_key_unit (pending_event, NULL, + NULL, NULL, &all_headers, &count); + } event = gst_video_event_new_downstream_force_key_unit (timestamp, stream_time, -- cgit v1.2.1