[MPlayer-dev-eng] [PATCH] fix memleak in mencoder.c

Pascal Massimino pascal.massimino at gmail.com
Sat May 3 19:39:11 CEST 2008


  Diego,

On Sat, May 3, 2008 at 8:45 AM, Diego Biurrun <diego at biurrun.de> wrote:
> On Sat, May 03, 2008 at 08:14:29AM -0700, Pascal Massimino wrote:
>  >
>  >     get_path()'s returned string needed to be freed.
>  >    Patch attached.
>  >
>  > --- mencoder.c        2008-05-03 07:32:05.139828000 -0700
>  > +++ mencoder.c        2008-05-03 07:32:05.143824000 -0700
>  > @@ -546,9 +546,13 @@
>  >         if(!vo_font)
>  > -       vo_font=read_font_desc(MPLAYER_DATADIR "/font/font.desc",font_factor,verbose>1);
>  > +         vo_font=read_font_desc(MPLAYER_DATADIR "/font/font.desc",font_factor,verbose>1);
>
>  This is an unrelated indentation fix.  I have committed it separately.

  thanks, sorry about that.
  Attached, an updated patch.

skal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mencoder_leak1-2.diff
Type: application/octet-stream
Size: 1317 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080503/d503edcf/attachment.obj>


More information about the MPlayer-dev-eng mailing list