[FFmpeg-cvslog] avcodec_decode_audio: do not trust the channel layout, use the channel count.

Michael Niedermayer git at videolan.org
Thu Feb 7 01:22:21 CET 2013


ffmpeg | branch: release/1.0 | Michael Niedermayer <michaelni at gmx.at> | Fri Jan 18 01:28:44 2013 +0100| [4cd1dad91ae97fe1f0dd534c3f5566787566f137] | committer: Michael Niedermayer

avcodec_decode_audio: do not trust the channel layout, use the channel count.

Fixes memory corruption

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
(cherry picked from commit d270c3202539e8364c46410e15f7570800e33343)

Conflicts:

	libavcodec/utils.c

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

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

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 00434c9..384d457 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -1744,7 +1744,7 @@ int attribute_align_arg avcodec_decode_audio4(AVCodecContext *avctx,
      * make sure it's set correctly; assume decoders that actually use
      * extended_data are doing it correctly */
     planar   = av_sample_fmt_is_planar(frame->format);
-    channels = av_get_channel_layout_nb_channels(frame->channel_layout);
+    channels = frame->channels;
     if (!(planar && channels > AV_NUM_DATA_POINTERS))
         frame->extended_data = frame->data;
 



More information about the ffmpeg-cvslog mailing list