[MPlayer-dev-eng] memory leak because of missing demux_close_asf()

[Guillaume POIRIER]

Hi,

On 10/1/06, Aurelien Jacobs <aurel at gnuage.org> wrote:
> On Sun, 01 Oct 2006 15:06:26 +0200
> Burkhard Fischer <b.fi2 at web.de> wrote:
>
> > Hi Guillaume,
> >
> > I tested my patch against the current SVN version. The memory leak still
> > exists.
>
> Please don't top-post.
>
> > Here is my patch (svn diff output):
> >
> > Index: libmpdemux/demux_asf.c
> > ===================================================================
> > --- libmpdemux/demux_asf.c    (revision 20011)
> > +++ libmpdemux/demux_asf.c    (working copy)
> > @@ -553,6 +553,8 @@
> >       //---- ASF header:
> >       if(!asf) return NULL;
> >       if (!read_asf_header(demuxer,asf)) {
> > +        if (asf->packet)
> > +            free(asf->packet);
> >           free(asf);
> >           return NULL;
> >       }
>
> Inconsistent indentation (same applies to your whole patch).
>
> Aurel

Other than that, is the patch ok? I can sure fix the identation before
commiting...

Guillaume

-- 
With DADVSI (http://en.wikipedia.org/wiki/DADVSI), France finally has
a lead on USA on selling out individuals right to corporations!
Vive la France!