On 2014-01-20 18:50:26 +0100, Anton Khirnov wrote:
> 
> On Mon, 20 Jan 2014 18:35:00 +0100, Janne Grunau <[email protected]> 
> wrote:
> > On 2014-01-20 15:27:46 +0100, Anton Khirnov wrote:
> > > The default get_buffer2() implementation (and possibly some
> > > user ones) does not allocate edges when this flag is set, which may
> > > expose bugs in some decoders. Until the 10 release is out, it is safer
> > > to remove this part.
> > > ---
> > >  libavcodec/utils.c |    5 -----
> > >  1 file changed, 5 deletions(-)
> > > 
> > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> > > index 2e418ec..044413a 100644
> > > --- a/libavcodec/utils.c
> > > +++ b/libavcodec/utils.c
> > > @@ -1091,11 +1091,6 @@ int attribute_align_arg 
> > > avcodec_open2(AVCodecContext *avctx, const AVCodec *code
> > >              ret = AVERROR(EINVAL);
> > >              goto free_and_end;
> > >          }
> > > -
> > > -#if FF_API_EMU_EDGE
> > > -        /* force the emu edge flag on, since it's now always active */
> > > -        avctx->flags |= CODEC_FLAG_EMU_EDGE;
> > > -#endif
> > >      }
> > >  end:
> > >      entangled_thread_counter--;
> > 
> > Fix for the h264 fate failure sent. Do you still want this committed as
> > protection against possible other bugs?
> 
> Yes, I'd like this in the release to be safe

patch ok

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to