[FFmpeg-devel] [PATCH] Get done with if (p) before av_free and av_freep

Clément Bœsch ubitux
Fri Feb 4 11:27:42 CET 2011


On Fri, Feb 04, 2011 at 10:11:37AM +0000, M?ns Rullg?rd wrote:
> Cl?ment B?sch <ubitux at gmail.com> writes:
> [...]
> > -    if( avctx->extradata != NULL )
> > -        av_freep(&avctx->extradata);
> > +    av_freep(&avctx->extradata);
> >      if( x->twopassbuffer != NULL ) {
> >          av_free(x->twopassbuffer);
> 
> That looks like another one right there.
> 

There is another av_free in this condition. It's not a trivial fix if
changed, I didn't want to include it in this patch if changed.

-- 
Cl?ment B.



More information about the ffmpeg-devel mailing list