[FFmpeg-cvslog] mpegaudiodec: fix short_start calculation

Luca Barbato git at videolan.org
Sat Sep 29 14:46:45 CEST 2012


ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Fri Sep 28 14:38:13 2012 +0200| [97cfa55eea39cef30abe14682c56c1e4e7f6f10d] | committer: Luca Barbato

mpegaudiodec: fix short_start calculation

The value should be always 3, as it follows from the specification.

Fix a stack buffer overflow in exponents_from_scale_factors as reported
by asan. Thanks to Dale Curtis for the sample vector.

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

 libavcodec/mpegaudiodec.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c
index 03094f6..ead0e1d 100644
--- a/libavcodec/mpegaudiodec.c
+++ b/libavcodec/mpegaudiodec.c
@@ -211,7 +211,7 @@ static void ff_compute_band_indexes(MPADecodeContext *s, GranuleDef *g)
             else
                 g->long_end = 6;
 
-            g->short_start = 2 + (s->sample_rate_index != 8);
+            g->short_start = 3;
         } else {
             g->long_end    = 0;
             g->short_start = 0;



More information about the ffmpeg-cvslog mailing list