On Mon, 4 Sep 2017 20:16:20 +0200 Michael Niedermayer <mich...@niedermayer.cc> wrote:
> On Mon, Sep 04, 2017 at 02:36:45PM -0300, James Almer wrote: > > 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. > > I find it a bit offensive that people suggest to remove the encoder i > maintain. Can I add my own unused fringe codec with no users, as long as I maintain it? _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel