virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-08-10 06:10:52 +02:00
Code Issues Releases Activity

avcodec/mpeg4videodec: Check sprite delta upshift against overflowing.

Browse Source 
Fixes: runtime error: signed integer overflow: -268386304 * 16 cannot be represented in type 'int'
Fixes: 2204/clusterfuzz-testcase-minimized-5616756909408256

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
Michael Niedermayer
2017-06-14 23:55:17 +02:00
parent 0a87be404a
commit 12245ab1f6
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
libavcodec/mpeg4videodec.c
View File

@@ -361,14 +361,16 @@ static int mpeg4_decode_sprite_trajectory(Mpeg4DecContext *ctx, GetBitContext *g
int shift_y = 16 - ctx->sprite_shift[0]; int shift_y = 16 - ctx->sprite_shift[0];
int shift_c = 16 - ctx->sprite_shift[1]; int shift_c = 16 - ctx->sprite_shift[1];
if (shift_c < 0 || shift_y < 0 || for (i = 0; i < 2; i++) {
FFABS(sprite_offset[0][0]) >= INT_MAX >> shift_y || if (shift_c < 0 || shift_y < 0 ||
FFABS(sprite_offset[1][0]) >= INT_MAX >> shift_c || FFABS( sprite_offset[0][i]) >= INT_MAX >> shift_y ||
FFABS(sprite_offset[0][1]) >= INT_MAX >> shift_y || FFABS( sprite_offset[1][i]) >= INT_MAX >> shift_c ||
FFABS(sprite_offset[1][1]) >= INT_MAX >> shift_c FFABS(s->sprite_delta[0][i]) >= INT_MAX >> shift_y ||
) { FFABS(s->sprite_delta[1][i]) >= INT_MAX >> shift_y
avpriv_request_sample(s->avctx, "Too large sprite shift or offset"); ) {
goto overflow; avpriv_request_sample(s->avctx, "Too large sprite shift, delta or offset");
goto overflow;
}
} }
for (i = 0; i < 2; i++) { for (i = 0; i < 2; i++) {