[FFmpeg-devel] tools/target_dec_fuzzer: Increase maxpixels threshold for dirac

Submitted by Michael Niedermayer on Aug. 24, 2019, 11 p.m.

Details

Message ID 20190824230030.17444-1-michael@niedermayer.cc
State Accepted
Commit 02a44ed0c8022da6c9c5bab7e5fb734dccefe587
Headers show

Commit Message

Michael Niedermayer Aug. 24, 2019, 11 p.m.
wavelets allow significant size expansion and they are also not
very fast.

Fixes: Timeout
Fixes: 16480/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DIRAC_fuzzer-5659892689403904 (108sec -> 17ms)
Fixes: 16480/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DIRAC_fuzzer-5749422717140992 (big ->15sec)

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 tools/target_dec_fuzzer.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Tomas Härdin Aug. 25, 2019, 6:35 a.m.
sön 2019-08-25 klockan 01:00 +0200 skrev Michael Niedermayer:
> wavelets allow significant size expansion and they are also not
> very fast.
> 
> Fixes: Timeout
> Fixes: 16480/clusterfuzz-testcase-minimized-
> ffmpeg_AV_CODEC_ID_DIRAC_fuzzer-5659892689403904 (108sec -> 17ms)
> Fixes: 16480/clusterfuzz-testcase-minimized-
> ffmpeg_AV_CODEC_ID_DIRAC_fuzzer-5749422717140992 (big ->15sec)
> 
> Found-by: continuous fuzzing process 
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  tools/target_dec_fuzzer.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c
> index 7bf25608ba..dd204eb5a6 100644
> --- a/tools/target_dec_fuzzer.c
> +++ b/tools/target_dec_fuzzer.c
> @@ -128,6 +128,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data,
> size_t size) {
>      maxpixels = maxpixels_per_frame * maxiteration;
>      switch (c->id) {
>          // Allows a small input to generate gigantic output
> +    case AV_CODEC_ID_DIRAC:     maxpixels /= 8192; break;
>      case AV_CODEC_ID_MSRLE:     maxpixels /= 16;  break;
>      case AV_CODEC_ID_QTRLE:     maxpixels /= 16;  break;
>      case AV_CODEC_ID_GIF:       maxpixels /= 16;  break;

Fine by me

/Tomas

Patch hide | download patch | download mbox

diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c
index 7bf25608ba..dd204eb5a6 100644
--- a/tools/target_dec_fuzzer.c
+++ b/tools/target_dec_fuzzer.c
@@ -128,6 +128,7 @@  int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
     maxpixels = maxpixels_per_frame * maxiteration;
     switch (c->id) {
         // Allows a small input to generate gigantic output
+    case AV_CODEC_ID_DIRAC:     maxpixels /= 8192; break;
     case AV_CODEC_ID_MSRLE:     maxpixels /= 16;  break;
     case AV_CODEC_ID_QTRLE:     maxpixels /= 16;  break;
     case AV_CODEC_ID_GIF:       maxpixels /= 16;  break;