diff options
| author | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2018-11-02 14:18:54 -0400 |
|---|---|---|
| committer | Sebastian Dröge <sebastian@centricular.com> | 2018-11-07 23:28:03 +0200 |
| commit | 0dcabd66fbe2973c00eb09772b0c4ef7ec28e6dd (patch) | |
| tree | 1447f00776b784356cf371206f613185fd033418 | |
| parent | 722e91173e7abe3cdc07eca048564fa194c2427e (diff) | |
| download | gstreamer-plugins-bad-0dcabd66fbe2973c00eb09772b0c4ef7ec28e6dd.tar.gz | |
h264parser: Stop considering 5 bytes NAL complete
This is only TRUE for SEQ_END / STREAM_END. Before this patch, passing
any valid 5 bytes of a NAL would make the parser pretend this NAL is
complete.
| -rw-r--r-- | gst-libs/gst/codecparsers/gsth264parser.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/gst-libs/gst/codecparsers/gsth264parser.c b/gst-libs/gst/codecparsers/gsth264parser.c index 73968db93..30d6aea49 100644 --- a/gst-libs/gst/codecparsers/gsth264parser.c +++ b/gst-libs/gst/codecparsers/gsth264parser.c @@ -1343,7 +1343,13 @@ gst_h264_parser_identify_nalu (GstH264NalParser * nalparser, gst_h264_parser_identify_nalu_unchecked (nalparser, data, offset, size, nalu); - if (res != GST_H264_PARSER_OK || nalu->size == 1) + if (res != GST_H264_PARSER_OK) + goto beach; + + /* The two NALs are exactly 1 byte size and are placed at the end of an AU, + * there is no need to wait for the following */ + if (nalu->type == GST_H264_NAL_SEQ_END || + nalu->type == GST_H264_NAL_STREAM_END) goto beach; off2 = scan_for_start_codes (data + nalu->offset, size - nalu->offset); |