* [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory @ 2003-08-04 15:36 ` Alexey Tourbin 2003-08-04 15:43 ` Sergey Bolshakov 2003-08-04 15:46 ` Dmitry V. Levin 0 siblings, 2 replies; 7+ messages in thread From: Alexey Tourbin @ 2003-08-04 15:36 UTC (permalink / raw) To: devel [-- 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 --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:36 ` [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory Alexey Tourbin @ 2003-08-04 15:43 ` Sergey Bolshakov 2003-08-04 15:46 ` Dmitry V. Levin 1 sibling, 0 replies; 7+ messages in thread From: Sergey Bolshakov @ 2003-08-04 15:43 UTC (permalink / raw) To: devel >>>>> "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, то ну его нафиг. -- ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:36 ` [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory Alexey Tourbin 2003-08-04 15:43 ` Sergey Bolshakov @ 2003-08-04 15:46 ` Dmitry V. Levin 2003-08-04 15:50 ` Alexander Bokovoy 2003-08-05 5:50 ` Dmitry V. Levin 1 sibling, 2 replies; 7+ messages in thread From: Dmitry V. Levin @ 2003-08-04 15:46 UTC (permalink / raw) To: ALT Devel discussion list [-- 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 --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:46 ` Dmitry V. Levin @ 2003-08-04 15:50 ` Alexander Bokovoy 2003-08-04 15:55 ` Dmitry V. Levin 2003-08-05 5:50 ` Dmitry V. Levin 1 sibling, 1 reply; 7+ messages in thread From: Alexander Bokovoy @ 2003-08-04 15:50 UTC (permalink / raw) To: ALT Devel discussion list 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. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:50 ` Alexander Bokovoy @ 2003-08-04 15:55 ` Dmitry V. Levin 2003-08-05 5:39 ` Alex Ott 0 siblings, 1 reply; 7+ messages in thread From: Dmitry V. Levin @ 2003-08-04 15:55 UTC (permalink / raw) To: ALT Devel discussion list [-- 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 --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:55 ` Dmitry V. Levin @ 2003-08-05 5:39 ` Alex Ott 0 siblings, 0 replies; 7+ messages in thread From: Alex Ott @ 2003-08-05 5:39 UTC (permalink / raw) To: devel >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 не живой. >Мы не сможем портироваться на такие древние платформы. >Так что лучше не перестраховываться. А потом мне искать почему программа валится :-) лучше перестраховаться. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory 2003-08-04 15:46 ` Dmitry V. Levin 2003-08-04 15:50 ` Alexander Bokovoy @ 2003-08-05 5:50 ` Dmitry V. Levin 1 sibling, 0 replies; 7+ messages in thread From: Dmitry V. Levin @ 2003-08-05 5:50 UTC (permalink / raw) To: ALT Devel discussion list [-- 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 --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2003-08-05 5:50 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2003-08-04 15:36 ` [devel] Re: [ALT Linux 0002810]: mpg123 -Z file eats LOTS of memory Alexey Tourbin 2003-08-04 15:43 ` Sergey Bolshakov 2003-08-04 15:46 ` Dmitry V. Levin 2003-08-04 15:50 ` Alexander Bokovoy 2003-08-04 15:55 ` Dmitry V. Levin 2003-08-05 5:39 ` Alex Ott 2003-08-05 5:50 ` Dmitry V. Levin
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git