avcodec/vp56: Add () to protect TRANSPOSE()s arguments
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 17 Feb 2015 17:57:26 +0000 (18:57 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 17 Feb 2015 18:13:26 +0000 (19:13 +0100)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavcodec/vp56.c

index 8571cf3..6319248 100644 (file)
@@ -701,7 +701,7 @@ av_cold int ff_vp56_init_context(AVCodecContext *avctx, VP56Context *s,
     ff_vp3dsp_init(&s->vp3dsp, avctx->flags);
     ff_vp56dsp_init(&s->vp56dsp, avctx->codec->id);
     for (i = 0; i < 64; i++) {
-#define TRANSPOSE(x) (x >> 3) | ((x & 7) << 3)
+#define TRANSPOSE(x) (((x) >> 3) | (((x) & 7) << 3))
         s->idct_scantable[i] = TRANSPOSE(ff_zigzag_direct[i]);
 #undef TRANSPOSE
     }