videoparsers: h265: Fix segfault while transforming hevc to nal aligned bytestream

Create temporary ParseFrame and copy the nal size buffer region
for each nal unit like we did for h264.

https://bugzilla.gnome.org/show_bug.cgi?id=763494
This commit is contained in:
Sreerenj Balachandran 2016-03-11 17:34:03 +02:00
parent 610705cbb4
commit 92f4eb45ab

View file

@ -762,12 +762,21 @@ gst_h265_parse_handle_frame_packetized (GstBaseParse * parse,
/* dispatch per NALU if needed */ /* dispatch per NALU if needed */
if (h265parse->split_packetized) { if (h265parse->split_packetized) {
GstBaseParseFrame tmp_frame;
gst_base_parse_frame_init (&tmp_frame);
tmp_frame.flags |= frame->flags;
tmp_frame.offset = frame->offset;
tmp_frame.overhead = frame->overhead;
tmp_frame.buffer = gst_buffer_copy_region (buffer, GST_BUFFER_COPY_ALL,
nalu.offset, nalu.size);
/* note we don't need to come up with a sub-buffer, since /* note we don't need to come up with a sub-buffer, since
* subsequent code only considers input buffer's metadata. * subsequent code only considers input buffer's metadata.
* Real data is either taken from input by baseclass or * Real data is either taken from input by baseclass or
* a replacement output buffer is provided anyway. */ * a replacement output buffer is provided anyway. */
gst_h265_parse_parse_frame (parse, frame); gst_h265_parse_parse_frame (parse, &tmp_frame);
ret = gst_base_parse_finish_frame (parse, frame, nl + nalu.size); ret = gst_base_parse_finish_frame (parse, &tmp_frame, nl + nalu.size);
left -= nl + nalu.size; left -= nl + nalu.size;
} }