On 9/4/2017 2:23 PM, wm4 wrote: > On Mon, 4 Sep 2017 18:11:44 +0200 > Michael Niedermayer <mich...@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 <mich...@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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
