[MPlayer-dev-eng] [PATCH] Libass BOM fix
ulion2002 at gmail.com
Sat Oct 27 18:05:33 CEST 2007
2007/10/27, Ivan Kalvachev <ikalvachev at gmail.com>:
> 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. ;)
Variable tmp may removed since it's not used any more.
More information about the MPlayer-dev-eng