[MPlayer-cvslog] CVS: main/libmpdemux demux_ogg.c,1.69,1.70

[Dominik 'Rathann' Mierzejewski]

On Monday, 02 May 2005 at 09:59, Reimar Döffinger CVS wrote:
> CVS change done by Reimar Döffinger CVS
> 
> Update of /cvsroot/mplayer/main/libmpdemux
> In directory mail:/var2/tmp/cvs-serv25400/libmpdemux
> 
> Modified Files:
> 	demux_ogg.c 
> Log Message:
> Fix the memleak fix: in case of error, demux_close_ogg should be called
> only once and demuxer->priv be freed.
> 
> 
> Index: demux_ogg.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libmpdemux/demux_ogg.c,v
> retrieving revision 1.69
> retrieving revision 1.70
> diff -u -r1.69 -r1.70
> --- demux_ogg.c	1 May 2005 10:40:42 -0000	1.69
> +++ demux_ogg.c	2 May 2005 07:59:12 -0000	1.70
> @@ -756,6 +756,7 @@
>    clear_sub = -1;
>    s = demuxer->stream;
>  
> +  demuxer->priv =
>    ogg_d = (ogg_demuxer_t*)calloc(1,sizeof(ogg_demuxer_t));
>    sync = &ogg_d->sync;
>    page = &ogg_d->page;

I haven't tried to compile this yet, but I'm pretty certain that this won't
compile without an initial value and a semicolon. Surely you meant
> +  demuxer->priv = ogg_d;
didn't you?

R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"