[MPlayer-dev-eng] [PATCH] fix a memleak on sub_font, it was never freed
Ulion
ulion2002 at gmail.com
Fri Jan 4 01:41:46 CET 2008
2008/1/4, Guillaume LECERF <foxcore at gmail.com>:
> 2008/1/3, Benjamin Zores <ben at geexbox.org>:
> > if (sub_font_name)
> > load_font_ft(dxs, dys, &sub_font, sub_font_name);
> > else
> > - sub_font = vo_font;
> > + load_font_ft(dxs, dys, &sub_font, font_name);
> >
> >
> > => load_font_ft(dxs, dyx, &sub_font, sub_font_name ? sub_font_name :
> > font_name);
>
> Patch updated.
This time you forget to set NULL after free in some places.
--
Ulion
More information about the MPlayer-dev-eng
mailing list