[FFmpeg-cvslog] avformat/iamf_parse: clear padding

Michael Niedermayer git at videolan.org
Sat Aug 31 20:58:35 EEST 2024


ffmpeg | branch: release/7.0 | Michael Niedermayer <michael at niedermayer.cc> | Wed Aug 14 16:34:19 2024 +0200| [7950855197cff106f62aa5a63ce22804ac9441c4] | committer: James Almer

avformat/iamf_parse: clear padding

Fixes: use of uninitialized value
Fixes: 70929/clusterfuzz-testcase-minimized-ffmpeg_dem_IAMF_fuzzer-5931276639469568

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Reviewed-by: James Almer <jamrial at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
(cherry picked from commit 7e5410eadb51645e67d91708494c7566771f9015)

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7950855197cff106f62aa5a63ce22804ac9441c4
---

 libavformat/iamf_parse.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavformat/iamf_parse.c b/libavformat/iamf_parse.c
index af19b31e70..3abbf70c8d 100644
--- a/libavformat/iamf_parse.c
+++ b/libavformat/iamf_parse.c
@@ -1080,6 +1080,7 @@ int ff_iamfdec_read_descriptors(IAMFContext *c, AVIOContext *pb,
         size = avio_read(pb, header, FFMIN(MAX_IAMF_OBU_HEADER_SIZE, max_size));
         if (size < 0)
             return size;
+        memset(header + size, 0, AV_INPUT_BUFFER_PADDING_SIZE);
 
         len = ff_iamf_parse_obu_header(header, size, &obu_size, &start_pos, &type, NULL, NULL);
         if (len < 0 || obu_size > max_size) {



More information about the ffmpeg-cvslog mailing list