From 7d1be36561616a9cc483a2fdd6b9e5f1cc1accb9 Mon Sep 17 00:00:00 2001 From: Nicolas Dufresne Date: Fri, 2 Nov 2018 15:53:56 -0400 Subject: [PATCH] h265parser: Stop considering 6 bytes NAL complete This is only TRUE for EOS / EOB NALs. Before this patch, passing any valid 6 bytes of a NAL would make the parser pretend this NAL is complete. --- gst-libs/gst/codecparsers/gsth265parser.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/gst-libs/gst/codecparsers/gsth265parser.c b/gst-libs/gst/codecparsers/gsth265parser.c index cf7f71f..d1dba60 100644 --- a/gst-libs/gst/codecparsers/gsth265parser.c +++ b/gst-libs/gst/codecparsers/gsth265parser.c @@ -1226,7 +1226,12 @@ gst_h265_parser_identify_nalu (GstH265Parser * parser, gst_h265_parser_identify_nalu_unchecked (parser, data, offset, size, nalu); - if (res != GST_H265_PARSER_OK || nalu->size == 2) + if (res != GST_H265_PARSER_OK) + goto beach; + + /* The two NALs are exactly 2 bytes size and are placed at the end of an AU, + * there is no need to wait for the following */ + if (nalu->type == GST_H265_NAL_EOS || nalu->type == GST_H265_NAL_EOB) goto beach; off2 = scan_for_start_codes (data + nalu->offset, size - nalu->offset); -- 2.7.4