Right, thanks.

Fed

On Mon, Apr 20, 2015 at 8:35 PM, Luca Barbato <[email protected]> wrote:

> On 20/04/15 21:12, Federico Tomassetti wrote:
> > This correction was suggested to me by Michael Niedermayer
> > ---
> >  libavfilter/buffersrc.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c
> > index f768063..8f7032f 100644
> > --- a/libavfilter/buffersrc.c
> > +++ b/libavfilter/buffersrc.c
> > @@ -195,7 +195,7 @@ do {
>                     \
> >      ref_out = av_buffer_create(data, data_size, compat_unref_buffer,
> \
> >                                 dummy_ref, 0);
>  \
> >      if (!ref_out) {
>  \
> > -        av_freep(&dummy_ref);
>  \
> > +        av_buffer_unref(&dummy_ref);
> \
> >          av_frame_unref(frame);
> \
> >          ret = AVERROR(ENOMEM);
> \
> >          goto fail;
> \
> >
>
> Seems fine to me, for similar patches you might also point the commit
> causing the problem, it is quite useful if the bug and the fix happened
> across two releases.
>
> lu
> _______________________________________________
> libav-devel mailing list
> [email protected]
> https://lists.libav.org/mailman/listinfo/libav-devel
>



-- 
Website at http://tomassetti.me <http://www.federico-tomassetti.it>
GitHub https://github.com/ftomassetti
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to