[FFmpeg-devel] [PATCH] avcodec/snowdec: Check intra block dc differences.
nfxjfg at googlemail.com
Mon Sep 4 21:35:17 EEST 2017
On Mon, 4 Sep 2017 20:16:20 +0200
Michael Niedermayer <michael at 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 <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.
> I find it a bit offensive that people suggest to remove the encoder i
Can I add my own unused fringe codec with no users, as long as I
More information about the ffmpeg-devel