[FFmpeg-devel] [PATCH 3/5] tools/target_dec_fuzzer: Adjust VP7 threshold

[Michael Niedermayer]

On Sun, Sep 29, 2019 at 03:34:33PM +1000, Peter Ross wrote:
> On Sun, Sep 29, 2019 at 01:53:43AM +0200, Michael Niedermayer wrote:
> > Fixes: Timeout (110sec -> 10sec)
> > Fixes: 17705/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP7_fuzzer-5765834135306240
> > 
> > Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> > Signed-off-by: Michael Niedermayer <michael at 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 4af7b26e54..a2c59be318 100644
> > --- a/tools/target_dec_fuzzer.c
> > +++ b/tools/target_dec_fuzzer.c
> > @@ -153,6 +153,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
> >      case AV_CODEC_ID_MSS2:        maxpixels /= 16384; break;
> >      case AV_CODEC_ID_SNOW:        maxpixels /= 128; break;
> >      case AV_CODEC_ID_TRUEMOTION2: maxpixels /= 1024; break;
> > +    case AV_CODEC_ID_VP7:         maxpixels /= 256;  break;
> >      }
> 
> ok. however i am curious why this is needed for vp7 and not vp8.

The CPU time was spend in vp7_fade_frame() i dont see a similar
call for vp8 but maybe the fuzzer will find something in vp8
too eventually

will apply

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190929/efb8772a/attachment.sig>