Prevent DAUD PCM encoder from fetching values outside of ff_reverse[] array bounds...
authorPeter Ross <pross@xvid.org>
Thu, 7 Aug 2008 08:16:42 +0000 (08:16 +0000)
committerPeter Ross <pross@xvid.org>
Thu, 7 Aug 2008 08:16:42 +0000 (08:16 +0000)
Originally committed as revision 14658 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/pcm.c

index d8b013b..3b61518 100644 (file)
@@ -209,7 +209,7 @@ static int pcm_encode_frame(AVCodecContext *avctx,
         break;
     case CODEC_ID_PCM_S24DAUD:
         for(;n>0;n--) {
-            uint32_t tmp = ff_reverse[*samples >> 8] +
+            uint32_t tmp = ff_reverse[(*samples >> 8) & 0xff] +
                            (ff_reverse[*samples & 0xff] << 8);
             tmp <<= 4; // sync flags would go here
             bytestream_put_be24(&dst, tmp);