[FFmpeg-devel] [PATCH 5/5] avformat/vividas: Check avio_read() for failure

Michael Niedermayer michael at niedermayer.cc
Sun Nov 3 22:12:16 EET 2024


Fixes: use of uninitialized value (untested)
Fixes: 42537627/clusterfuzz-testcase-minimized-fuzzer_loadfile_direct-5505802505355264

Found-by: ossfuzz
Reported-by: Kacper Michajlow
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 libavformat/vividas.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavformat/vividas.c b/libavformat/vividas.c
index 130b81ebbe1..bec4ad7cec0 100644
--- a/libavformat/vividas.c
+++ b/libavformat/vividas.c
@@ -567,7 +567,8 @@ static int viv_read_header(AVFormatContext *s)
     v = avio_r8(pb);
     avio_seek(pb, v, SEEK_CUR);
 
-    avio_read(pb, keybuffer, 187);
+    if (avio_read(pb, keybuffer, 187) != 187)
+        return AVERROR_INVALIDDATA;
     key = decode_key(keybuffer);
     viv->sb_key = key;
 
-- 
2.47.0



More information about the ffmpeg-devel mailing list