On 2/25/17, Michael Niedermayer <mich...@niedermayer.cc> wrote: > Fixes: 673/clusterfuzz-testcase-5948736536576000 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavcodec/flacdsp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/flacdsp.c b/libavcodec/flacdsp.c > index 560091f73a..bc9a5dbed9 100644 > --- a/libavcodec/flacdsp.c > +++ b/libavcodec/flacdsp.c > @@ -67,7 +67,7 @@ static void flac_lpc_16_c(int32_t *decoded, const int > coeffs[32], > int sum = 0; > for (j = 0; j < pred_order; j++) > sum += coeffs[j] * (SUINT)decoded[j]; > - decoded[j] += sum >> qlevel; > + decoded[j] = decoded[j] + (unsigned)(sum >> qlevel); > } > } >
What about making sum unsigned instead? _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
