[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