[FFmpeg-devel] [PATCH] avcodec/snowdec: Check intra block dc differences.

James Almer jamrial at gmail.com
Mon Sep 4 20:36:45 EEST 2017


On 9/4/2017 2:23 PM, wm4 wrote:
> On Mon,  4 Sep 2017 18:11:44 +0200
> Michael Niedermayer <michael at niedermayer.cc> wrote:
> 
>> Fixes: Timeout
>> Fixes: 3142/clusterfuzz-testcase-5007853163118592
>>
>> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> 
> Wasn't this patch rejected before?

Ronald rejected the previous version because of the obscure error
messages. I suggested an alternative and Michael sent a new patch
following it.

If this is also not ok then please say so. That way the error messages
can be removed before pushing the patch and we can go back to focus on
things worth discussing.

> 
> Can't we just remove this codec? It has no use other than causing
> potential security issues and maintenance.

I agree about removing the encoder, but the decoder is needed for
existing streams. Unless everyone just argues it has no real world
samples for being an avcodec-only toy codec.

If you send a patch removing the encoder without also trying to remove
all the things that currently depend on it (One or two filters i think)
then I'll +1 it. ffv1 and mpeg4 encoders are enough for any kind of
testing, fate or otherwise, that might require a native encoder.


More information about the ffmpeg-devel mailing list