[MPlayer-dev-eng] [PATCH][BUG] Incorrect memleak fix code in input/input.c might cause incorrect free
Alexander Strasser
eclipse7 at gmx.net
Sat Jul 31 12:24:48 CEST 2004
Hi Shachar,
On Mon, Jul 26, 2004 at 05:59:42PM +0200, Shachar Raindel wrote:
> > Just a view things:
> > > if( mp_input_parse_config(file)) {
> > > - free(file); // release the buffer created by get_path()
> > > + // free(file); // release the buffer created by get_path()
> > > }
> > > else {
> > This doesn't look to nice, but maybe i'm blind again.
> > I would suggest:
> > if( !mp_input_parse_config(file)) {
> >
> > > + free(tmpbufptr);
> > Maybe the freeing comment should be appended here.
> >
> > Alex (beastd)
> >
>
> I agree. The patch was written under time presure, and therefore I
> hadn't had time to make it pretty. I should clean-up my tree at home
> soon, and send some cleanup patches, so that I will not have so much
> CVS merge problems....
Do you want to resend a cleaned up patch?
If not i'm going to send one and apply it asap,
because I want to correct my mistake asap.
Alex (beastd)
More information about the MPlayer-dev-eng
mailing list