Re: [FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-10-16 Thread Michael Niedermayer
On Wed, Oct 16, 2019 at 06:27:10PM +0200, Paul B Mahol wrote: > On 10/16/19, Michael Niedermayer wrote: > > On Thu, Sep 26, 2019 at 11:18:37PM +0200, Michael Niedermayer wrote: > >> On Thu, Sep 26, 2019 at 10:04:44AM +0200, Paul B Mahol wrote: > >> > Why this does not set dirty for all decoding

Re: [FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-10-16 Thread Paul B Mahol
On 10/16/19, Michael Niedermayer wrote: > On Thu, Sep 26, 2019 at 11:18:37PM +0200, Michael Niedermayer wrote: >> On Thu, Sep 26, 2019 at 10:04:44AM +0200, Paul B Mahol wrote: >> > Why this does not set dirty for all decoding cases? >> >> dirty is set for c->current_frame, so cases which write

Re: [FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-10-16 Thread Michael Niedermayer
On Thu, Sep 26, 2019 at 11:18:37PM +0200, Michael Niedermayer wrote: > On Thu, Sep 26, 2019 at 10:04:44AM +0200, Paul B Mahol wrote: > > Why this does not set dirty for all decoding cases? > > dirty is set for c->current_frame, so cases which write only > into that do not need an explicit case. >

Re: [FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-09-26 Thread Michael Niedermayer
On Thu, Sep 26, 2019 at 10:04:44AM +0200, Paul B Mahol wrote: > Why this does not set dirty for all decoding cases? dirty is set for c->current_frame, so cases which write only into that do not need an explicit case. But maybe i forgot something ? Thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-09-26 Thread Paul B Mahol
Why this does not set dirty for all decoding cases? On 9/25/19, Michael Niedermayer wrote: > This avoids unneeded operations and makes the code faster. > > Fixes: Timeout > Fixes: > 15724/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PAF_VIDEO_fuzzer-5750842205929472 > (12sec -> 9sec) >

[FFmpeg-devel] [PATCH 09/12] avcodec/pafvideo: Only clear frame when it was written to

2019-09-25 Thread Michael Niedermayer
This avoids unneeded operations and makes the code faster. Fixes: Timeout Fixes: 15724/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PAF_VIDEO_fuzzer-5750842205929472 (12sec -> 9sec) Fixes: 17625/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PAF_VIDEO_fuzzer-5640515311108096 (16sec ->