[FFmpeg-devel] [PATCH 2/6] avcodec/dca_xll: Fix runtime error: signed integer overflow: 2147286116 + 6298923 cannot be represented in type 'int'

wm4 nfxjfg at googlemail.com
Thu Mar 9 09:00:03 EET 2017


On Wed,  8 Mar 2017 23:28:39 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> Fixes: 732/clusterfuzz-testcase-4872990070145024
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/dca_xll.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c
> index 6cebda35e4..551774ebc6 100644
> --- a/libavcodec/dca_xll.c
> +++ b/libavcodec/dca_xll.c
> @@ -1312,7 +1312,7 @@ static int combine_residual_frame(DCAXllDecoder *s, DCAXllChSet *c)
>          } else {
>              // No downmix scaling
>              for (n = 0; n < nsamples; n++)
> -                dst[n] += (src[n] + round) >> shift;
> +                dst[n] += (SUINT)((src[n] + round) >> shift);
>          }
>      }
>  

I'm kind of against this SUINT stuff spreading.

Changing the sign depending on debug mode? Sounds horrible.


More information about the ffmpeg-devel mailing list