From cedc7d4def9860d3fba2884a7a44f827e9903f68 Mon Sep 17 00:00:00 2001 From: Sreerenj Balachandran Date: Tue, 30 Jan 2018 11:34:58 -0900 Subject: [PATCH] codecparser: vp8: Fix range decoder init According to the vp8 spec, the first partition (size can be derived from the frame header) should have all compressed header information and we implemented gst codecparser based on that. But it doesn't seem to be the case with some of the streams (#792773) and libvpx works fine because it uses the whole frame size (not the first partition size) to initialize the bool decoder. https://bugzilla.gnome.org/show_bug.cgi?id=792773 --- gst-libs/gst/codecparsers/gstvp8parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gst-libs/gst/codecparsers/gstvp8parser.c b/gst-libs/gst/codecparsers/gstvp8parser.c index ee91858..c3345e5 100644 --- a/gst-libs/gst/codecparsers/gstvp8parser.c +++ b/gst-libs/gst/codecparsers/gstvp8parser.c @@ -548,7 +548,7 @@ gst_vp8_parser_parse_frame_header (GstVp8Parser * parser, data += frame_hdr->data_chunk_size; size -= frame_hdr->data_chunk_size; - if (!gst_vp8_range_decoder_init (&rd, data, frame_hdr->first_part_size)) + if (!gst_vp8_range_decoder_init (&rd, data, size)) return GST_VP8_PARSER_BROKEN_DATA; result = parse_frame_header (parser, &rd, frame_hdr); -- 2.7.4