Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory

[Alexander Bokovoy <a.bokovoy@sam-solutions.net>]

On Mon, Aug 04, 2003 at 07:46:47PM +0400, Dmitry V. Levin wrote:
> On Mon, Aug 04, 2003 at 07:36:01PM +0400, Alexey Tourbin wrote:
> > On Mon, Aug 04, 2003 at 06:18:35PM +0400, mantis-admin@altlinux.ru wrote:
> > > =======================================================================
> > > Summary:                    mpg123 -Z file eats LOTS of memory
> > > Description: 
> > > in situation where file is not a mp3, e.g.
> > > mpg123 -Z ~/.bash_profile
> > 
> > Зафиксил.  Ждите новой сборки на этой неделе.  Стоит ли включить
> > поддержку esound?  Когда-то она была, но goldhead её выключил или просто
> > разломал (там кривоватый Makefile).
> > 
> > --- mpg123/readers.c~	2000-10-30 20:12:59 +0000
> > +++ mpg123/readers.c	2003-08-04 15:26:46 +0000
> > @@ -185,6 +185,8 @@ void stream_close(struct reader *rds)
> >  {
> >      if (rds->flags & READER_FD_OPENED)
> >          close(rds->filept);
> > +    if (rds->backbuf)
> > +	    free(rds->backbuf);
> >  }
> >  
> >  /**************************************** 
> 
> if (data) free (data)
> - это масло маслянное.
> пишите просто
> free (data)
> - все соответствующие стандарту libc это поддерживают.
Solaris < 8 валится. Лучше перестраховаться.

-- 
/ Alexander Bokovoy
---
Must be getting close to town -- we're hitting more people.