[FFmpeg-devel] [PATCH] avutil/integer: Fix integer overflow in av_mul_i()

Michael Niedermayer michael at niedermayer.cc
Wed Oct 24 03:01:36 EEST 2018


Found-by: fate
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 libavutil/integer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavutil/integer.c b/libavutil/integer.c
index 890e314dce..78e252fbde 100644
--- a/libavutil/integer.c
+++ b/libavutil/integer.c
@@ -74,7 +74,7 @@ AVInteger av_mul_i(AVInteger a, AVInteger b){
 
         if(a.v[i])
             for(j=i; j<AV_INTEGER_SIZE && j-i<=nb; j++){
-                carry= (carry>>16) + out.v[j] + a.v[i]*b.v[j-i];
+                carry= (carry>>16) + out.v[j] + a.v[i]*(unsigned)b.v[j-i];
                 out.v[j]= carry;
             }
     }
-- 
2.19.1



More information about the ffmpeg-devel mailing list