[FFmpeg-cvslog] segafilm: Validate the number of audio channels

Martin Storsjö git at videolan.org
Thu Jan 16 22:08:51 CET 2014


ffmpeg | branch: release/0.10 | Martin Storsjö <martin at martin.st> | Thu Sep 19 16:02:29 2013 +0300| [e80071892b14a66b8730dfe21aca76e5b0333f2d] | committer: Luca Barbato

segafilm: Validate the number of audio channels

This avoids divisions by zero later.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable at libav.org
Signed-off-by: Martin Storsjö <martin at martin.st>
(cherry picked from commit 82e266c6d3fbf3cc74e515b883e66543381a0f2c)
Signed-off-by: Luca Barbato <lu_zero at gentoo.org>
(cherry picked from commit 5379c5184b9fe9ef06234638f5629d4c80056e04)

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

 libavformat/segafilm.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavformat/segafilm.c b/libavformat/segafilm.c
index 5279121..d5aaf11 100644
--- a/libavformat/segafilm.c
+++ b/libavformat/segafilm.c
@@ -112,6 +112,11 @@ static int film_read_header(AVFormatContext *s,
             return AVERROR(EIO);
         film->audio_samplerate = AV_RB16(&scratch[24]);
         film->audio_channels = scratch[21];
+        if (!film->audio_channels || film->audio_channels > 2) {
+            av_log(s, AV_LOG_ERROR,
+                   "Invalid number of channels: %d\n", film->audio_channels);
+            return AVERROR_INVALIDDATA;
+        }
         film->audio_bits = scratch[22];
         if (scratch[23] == 2)
             film->audio_type = CODEC_ID_ADPCM_ADX;



More information about the ffmpeg-cvslog mailing list