On Fri, May 16, 2014 at 10:55:44AM -0400, Vittorio Giovara wrote:
> On Thu, May 15, 2014 at 2:29 AM, Anton Khirnov <[email protected]> wrote:
> > ---
> >  libavformat/a64.c |    6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/libavformat/a64.c b/libavformat/a64.c
> > index 640e786..da8e310 100644
> > --- a/libavformat/a64.c
> > +++ b/libavformat/a64.c
> > @@ -35,6 +35,12 @@ static int a64_write_header(AVFormatContext *s)
> >          0x00, //charset_lifetime (multi only)
> >          0x00  //fps in 50/fps;
> >      };
> > +
> > +    if (avctx->extradata_size < 4) {
> > +        av_log(s, AV_LOG_ERROR, "Missing extradata\n");
> > +        return AVERROR(EINVAL);
> > +    }
> > +
> 
> Shouldn't this be < 4 + FF_INPUT_BUFFER_PADDING_SIZE ?

Of course not. FF_INPUT_BUFFER_PADDING_SIZE is used only when allocating
extradata.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to