[Mplayer-cvslog] CVS: main/libmpdemux demux_ogg.c,1.15,1.16

Arpi arpi at thot.banki.hu
Thu Oct 17 02:18:06 CEST 2002


Hi,

> Log Message:
> more memleak fixes
> 
> Index: demux_ogg.c
> ===================================================================

>      if(np < 0) {
>        mp_msg(MSGT_DEMUX,MSGL_DBG2,"OGG demuxer : Bad page sync\n");
> +      ogg_sync_destroy(sync);
>        free(ogg_d);
>        return 0;
>      }
> @@ -425,6 +426,7 @@
>        buf = ogg_sync_buffer(sync,BLOCK_SIZE);
>        len = stream_read(s,buf,BLOCK_SIZE);      
>        if(len == 0 && s->eof) {
> +        ogg_sync_destroy(sync);
>  	free(ogg_d);


had to reverse this one, caused random memory corruption :(

does anyone know how to properly close/free ogg stream?
it leaks around 200kb per instance, for buffers.
i don't even know why do we need to init libogg for .ogm file detection,
afaik it has unique file signature at the begining.


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu



More information about the MPlayer-cvslog mailing list