[FFmpeg-cvslog] avcodec/h2645_parse: Ignore NAL with nuh_layer_id == 63
Michael Niedermayer
git at videolan.org
Mon Dec 2 04:28:05 EET 2024
ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Mon Nov 18 03:22:27 2024 +0100| [360e7cafd0e65fdf4b186c95e2517a94b9f3fa4f] | committer: Michael Niedermayer
avcodec/h2645_parse: Ignore NAL with nuh_layer_id == 63
Comply with "For purposes other than determining the amount of data in the decoding units
of the bitstream, decoders shall ignore all data that follow the value 63 for nuh_layer_id in a NAL unit"
Rec. ITU-T H.265 v8 (08/2021) Page 67
Fixes: index 63 out of bounds for type 'const int8_t[63]' (aka 'const signed char[63]')
Fixes: clusterfuzz-testcase-fuzzer_loadfile-5109286752026624
Reported-by: Kacper Michajlow <kasper93 at gmail.com>
Found-by: ossfuzz
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=360e7cafd0e65fdf4b186c95e2517a94b9f3fa4f
---
libavcodec/h2645_parse.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c
index 7b48fcae17..82816999e8 100644
--- a/libavcodec/h2645_parse.c
+++ b/libavcodec/h2645_parse.c
@@ -581,9 +581,11 @@ int ff_h2645_packet_split(H2645Packet *pkt, const uint8_t *buf, int length,
if (codec_id == AV_CODEC_ID_VVC)
ret = vvc_parse_nal_header(nal, logctx);
- else if (codec_id == AV_CODEC_ID_HEVC)
+ else if (codec_id == AV_CODEC_ID_HEVC) {
ret = hevc_parse_nal_header(nal, logctx);
- else
+ if (nal->nuh_layer_id == 63)
+ continue;
+ } else
ret = h264_parse_nal_header(nal, logctx);
if (ret < 0) {
av_log(logctx, AV_LOG_WARNING, "Invalid NAL unit %d, skipping.\n",
More information about the ffmpeg-cvslog
mailing list