[FFmpeg-devel] [PATCH 2/2] avcodec/lcldec: Check mthread_inlen instead of cliping

[Michael Niedermayer]

On Sun, Jul 28, 2019 at 12:31:22AM +0200, Michael Niedermayer wrote:
> Clipping was added in 2009 to avoid crashes.
> The clipped case would produce a 2nd slice with 0 input
> thus also producing 0 output.
> Subsequent checks will cause decoder failure unless both
> slices have the same output length. thus the only way this
> would not already fail is if the output from both slices
> was 0 bytes.
> 
> Fixes: Timeout (134sec -> 241ms)
> Fixes: 15599/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MSZH_fuzzer-5658127116009472
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/lcldec.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

will apply

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- 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/20190823/dae819b5/attachment.sig>