[MPlayer-dev-eng] [PATCH] mkv: remove ->sub->sh malloc hack and possible memleak

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Jun 24 09:29:11 CEST 2007


Hello,
On Sat, Jun 23, 2007 at 10:26:03PM +0200, Aurelien Jacobs wrote:
> On Sat, 23 Jun 2007 19:25:46 +0200
> Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> > this patch removes a hack that mallocs demuxer->sub->sh and memcpys the
> > contents over instead of just setting it to some demuxer->s_streams.
> > This reduces the size of the track struct, probably should fix a memleak
> > (though valgrind can't find any with the old code, but neither can it
> > find a double free with the new code... Hm, still some bugs left to
> > fix) and should allow to get rid of the special-case code for mkv track
> > switching.
> > Ok to apply?
> 
> Looks OK. (but I only gave it a very quick look)

Applied. I am around to fix any problem should there be any anyway.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list