Re: [CHECKER] 4 memory leaks in Postgresql 7.4.2

From: "Ted Kremenek" <kremenek(at)cs(dot)stanford(dot)edu>
To: "'Michael Meskes'" <meskes(at)postgresql(dot)org>, "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "'Neil Conway'" <neilc(at)samurai(dot)com>, <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: [CHECKER] 4 memory leaks in Postgresql 7.4.2
Date: 2004-07-08 17:50:48
Message-ID: 200407081751.i68Hp4qs006615@smtp-roam.Stanford.EDU
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Thank you for the confirmation.

Best,
Ted

-----Original Message-----
From: Michael Meskes [mailto:meskes(at)postgresql(dot)org]
Sent: Monday, July 05, 2004 11:55 AM
To: Tom Lane
Cc: Neil Conway; Ted Kremenek; pgsql-bugs(at)postgresql(dot)org; Michael Meskes
Subject: Re: [BUGS] [CHECKER] 4 memory leaks in Postgresql 7.4.2

On Sun, May 02, 2004 at 07:50:46PM -0400, Tom Lane wrote:
> >> [BUG] memory leak on error path (dtype != DTK_DELTA)
> >> File where bug occurred:
> >> postgresql-7.4.2/src/interfaces/ecpg/pgtypeslib/interval.c
>
> > Looks suspicious to me, but ECPG is Michael Meskes' domain -- Michael?
>
> It's entirely likely that ecpg's derivative of the backend's datetime
> modules contains lots and lots of memory leaks, since AFAIK the palloc
> infrastructure is not there in the ecpg environment :-(.

This is true of course. But I'm hopeful we found most of the possible
leaks. This one certainly was not found.

Fix just committed.

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes(at)jabber(dot)org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Markus Bertheau 2004-07-09 14:04:30 compile error in contrib/xml2
Previous Message Oliver Elphick 2004-07-08 17:23:01 Re: 7.4: serial not working ?

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2004-07-08 19:07:51 Re: working on support triggers on columns
Previous Message Klaus Naumann 2004-07-08 17:40:57 Update pg_tables, pg_indexes views to show tablespace name?