[FFmpeg-devel] [PATCH] errant pointer check in vhook/ppm.c

Diego Biurrun diego
Mon Jul 21 07:47:16 CEST 2008


On Thu, Jul 17, 2008 at 09:17:46PM +0100, M?ns Rullg?rd wrote:
> Erik Hovland <erik at hovland.org> writes:
> 
> > I recognize that the vhook stuff is deprecated. But since it still gets
> > compiled in trunk I figured I would submit this fix.
> >
> > The function Configure in vhook/ppm.c checks the given pointer and not
> > the allocated pointer because of a minor typo.
> >
> > --- a/vhook/ppm.c
> > +++ b/vhook/ppm.c
> > @@ -211,7 +211,7 @@ int Configure(void **ctxp, int argc, char *argv[])
> >          *ctxp = av_mallocz(sizeof(ContextInfo));
> > -        if ( ctxp != NULL && argc > 1 )
> > +        if ( *ctxp != NULL && argc > 1 )
> >          {
> 
> Patch looks OK.

Applied.

Diego




More information about the ffmpeg-devel mailing list