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

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

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 801 bytes --]

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);
 }
 
 /**************************************** 

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

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

[Sergey Bolshakov]

>>>>> "Alexey" == Alexey Tourbin <at@altlinux.ru> writes:

 > 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);
 >  }

Толпа кричала: "Чудо! Чудо!" :)
По мне, так эта прелесть вполне кандидат в updates для M2.2,
благо там она и была обнаружена.

Что до esound, то ну его нафиг.

-- 

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1046 bytes --]

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 это поддерживают.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

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

[Alexander Bokovoy]

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.

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 452 bytes --]

On Mon, Aug 04, 2003 at 06:50:26PM +0300, Alexander Bokovoy wrote:
> On Mon, Aug 04, 2003 at 07:46:47PM +0400, Dmitry V. Levin wrote:
> > if (data) free (data)
> > - это масло маслянное.
> > пишите просто
> > free (data)
> > - все соответствующие стандарту libc это поддерживают.
> Solaris < 8 валится. Лучше перестраховаться.

Solaris < 8 не живой.
Мы не сможем портироваться на такие древние платформы.
Так что лучше не перестраховываться.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

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

[Alex Ott]

>On Mon, Aug 04, 2003 at 06:50:26PM +0300, Alexander Bokovoy wrote:
>> On Mon, Aug 04, 2003 at 07:46:47PM +0400, Dmitry V. Levin wrote:
>> > if (data) free (data)
>> > - это масло маслянное.
>> > пишите просто
>> > free (data)
>> > - все соответствующие стандарту libc это поддерживают.
>> Solaris < 8 валится. Лучше перестраховаться.
>
>Solaris < 8 не живой.
>Мы не сможем портироваться на такие древние платформы.
>Так что лучше не перестраховываться.

А потом мне искать почему программа валится :-) лучше перестраховаться. 

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 305 bytes --]

On Mon, Aug 04, 2003 at 07:46:47PM +0400, Dmitry V. Levin wrote:
> if (data) free (data)
> - это масло маслянное.
> пишите просто
> free (data)
> - все соответствующие стандарту libc это поддерживают.

А ещё лучше
free (data);
data = 0;
- меньше шансов получить double free в кривых программах.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]