]> git.sesse.net Git - ffmpeg/commitdiff
avcodec/ac3: Explicitly return to discard large amounts of nonsense bytes
authorMichael Niedermayer <michael@niedermayer.cc>
Wed, 30 Jan 2019 23:45:51 +0000 (00:45 +0100)
committerMichael Niedermayer <michael@niedermayer.cc>
Wed, 6 Feb 2019 23:29:01 +0000 (00:29 +0100)
Changes 19sec to 10ms (12559) runtime, 17sec to 177ms (12570)
Fixes: Timeout
Fixes: 12559/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AC3_fuzzer-5666516266123264
Fixes: 12561/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AC3_FIXED_fuzzer-5682923041193984
Fixes: 12570/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EAC3_fuzzer-5194734308425728
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
libavcodec/ac3dec.c

index f844a463ee6baf72a9c2efbc7303be4ffe459f38..eaa327a3ee9446d9d59d6fcff99639699245fee1 100644 (file)
@@ -1490,6 +1490,8 @@ static int ac3_decode_frame(AVCodecContext * avctx, void *data,
     }
     if (i >= buf_size)
         return AVERROR_INVALIDDATA;
+    if (i > 10)
+        return i;
     buf += i;
     buf_size -= i;