Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-09 Thread Michael Niedermayer
On Sat, Sep 09, 2017 at 03:11:00PM +0100, Derek Buitenhuis wrote: > On 9/8/2017 11:15 PM, James Almer wrote: > > It reads eight bytes at a time if the buffer is sufficiently aligned, > > then finishes reading the remaining bytes one at a time. > > If the buffer is unaligned, it reads everything one

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-09 Thread Derek Buitenhuis
On 9/8/2017 11:15 PM, James Almer wrote: > It reads eight bytes at a time if the buffer is sufficiently aligned, > then finishes reading the remaining bytes one at a time. > If the buffer is unaligned, it reads everything one byte at a time like > it used to. > > See ff_h2645_extract_rbsp() and ad

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread James Almer
On 9/8/2017 8:49 PM, Michael Niedermayer wrote: > On Fri, Sep 08, 2017 at 06:43:06PM -0300, James Almer wrote: >> On 9/8/2017 6:29 PM, Michael Niedermayer wrote: >>> Speeds code up from 50sec to 15sec >>> >>> Fixes Timeout >>> Fixes: 3242/clusterfuzz-testcase-5811951672229888 >>> >>> Found-by: cont

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread Michael Niedermayer
On Fri, Sep 08, 2017 at 06:43:06PM -0300, James Almer wrote: > On 9/8/2017 6:29 PM, Michael Niedermayer wrote: > > Speeds code up from 50sec to 15sec > > > > Fixes Timeout > > Fixes: 3242/clusterfuzz-testcase-5811951672229888 > > > > Found-by: continuous fuzzing process > > https://github.com/go

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread James Almer
On 9/8/2017 6:47 PM, Kieran Kunhya wrote: > On Fri, 8 Sep 2017 at 22:29 Michael Niedermayer > wrote: > >> Speeds code up from 50sec to 15sec >> >> Fixes Timeout >> Fixes: 3242/clusterfuzz-testcase-5811951672229888 >> >> Found-by: continuous fuzzing process >> https://github.com/google/oss-fuzz/tr

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread Kieran Kunhya
On Fri, 8 Sep 2017 at 22:29 Michael Niedermayer wrote: > Speeds code up from 50sec to 15sec > > Fixes Timeout > Fixes: 3242/clusterfuzz-testcase-5811951672229888 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by >

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread James Almer
On 9/8/2017 6:29 PM, Michael Niedermayer wrote: > Speeds code up from 50sec to 15sec > > Fixes Timeout > Fixes: 3242/clusterfuzz-testcase-5811951672229888 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermaye

[FFmpeg-devel] [PATCH 3/3] avcodec/scpr: optimize shift loop.

2017-09-08 Thread Michael Niedermayer
Speeds code up from 50sec to 15sec Fixes Timeout Fixes: 3242/clusterfuzz-testcase-5811951672229888 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/scpr.c | 11 ++- 1 file changed, 10 i