[MPlayer-dev-eng] [PATCH] Libass BOM fix

Ivan Kalvachev ikalvachev at gmail.com
Sat Oct 27 17:19:49 CEST 2007


2007/10/27, Ulion <ulion2002 at gmail.com>:
> 2007/10/27, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> > Hello,
> > On Sat, Oct 27, 2007 at 10:01:29PM +0800, Ulion wrote:
> > > A memory leak bug found in sub_recode, my fix is here:
> >
> > Better don't send patches for new issues as replies.
> > Also if you send them inline take care your mailer doesn't wrap lines.
>
> Sorry for that. I sent it inline to be noticed directly, it's just a
> notifaction.
>
> > And attached patch seems simpler (and would allow to remove the later
> > #ifdef HAVE_ENCA).
> > Note that it is untested though.
>
> Your one is simper for resolve this little leak.
> But as ivan suggested, maybe we should get rid of all strdup stuff for
> enca, I think we should wait for his patch.

Here it is.
Have in mind there is something fishy with it. It compiled and run
from the first time.
According to Murphy's law, it means that the concept is flowed. ;)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: enca_const.patch
Type: application/octet-stream
Size: 3921 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071027/318471cd/attachment.obj>


More information about the MPlayer-dev-eng mailing list