Re: [HACKERS] leaks in TopMemoryContext?

2006-01-11 Thread Neil Conway
On Wed, 2006-01-11 at 02:58 -0500, Tom Lane wrote:
 One comment is that there are worse things than small memory leaks in
 seldom-followed code paths, especially if those paths are only taken in
 error cases.

While I agree the problem isn't a showstopper, I think it is still
worthy of concern: the mbutils example was chosen for being clearly
broken, not as being the most serious instance of the problem. The issue
might occur in *any* situation in which we're allocating memory in a
memory context whose lifetime exceeds the current transaction -- I
haven't looked to see what other places might need fixing.

-Neil



---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


Re: [HACKERS] leaks in TopMemoryContext?

2006-01-11 Thread Tom Lane
Neil Conway [EMAIL PROTECTED] writes:
 While I agree the problem isn't a showstopper, I think it is still
 worthy of concern: the mbutils example was chosen for being clearly
 broken, not as being the most serious instance of the problem. The issue
 might occur in *any* situation in which we're allocating memory in a
 memory context whose lifetime exceeds the current transaction -- I
 haven't looked to see what other places might need fixing.

There's very little code that runs in TopMemoryContext (by design), so
I don't think you'll find much.

regards, tom lane

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [HACKERS] leaks in TopMemoryContext?

2006-01-10 Thread Tom Lane
Neil Conway [EMAIL PROTECTED] writes:
 Thoughts?

One comment is that there are worse things than small memory leaks in
seldom-followed code paths, especially if those paths are only taken in
error cases.  I'm interested in fixing this if it can be done without
unreasonable code complication, but there are limits to how far we
should go to do it.

regards, tom lane

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings