From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Stephen Frost <sfrost(at)snowman(dot)net> |
Cc: | Ants Aasma <ants(at)cybertec(dot)at>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: Missing pfree in logical_heap_rewrite_flush_mappings() |
Date: | 2014-03-26 18:26:27 |
Message-ID: | 20140326182627.GE9066@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2014-03-26 13:41:27 -0400, Stephen Frost wrote:
> * Andres Freund (andres(at)2ndquadrant(dot)com) wrote:
> > On 2014-03-26 12:49:41 -0400, Stephen Frost wrote:
> > > * Ants Aasma (ants(at)cybertec(dot)at) wrote:
> > > > It seems to me that when flushing logical mappings to disk, each
> > > > mapping file leaks the buffer used to pass the mappings to XLogInsert.
> > > > Also, it seems consistent to allocate that buffer in the RewriteState
> > > > memory context. Patch attached.
> >
> > Good catch. There's actually no need for explicitly using the context,
> > we're in the appropriate one. The only other MemoryContextAlloc() caller
> > in there should be converted to a palloc as well.
>
> Hrm..? I don't think that's right when it's called from
> end_heap_rewrite().
You're right. Apprently I shouldn't look at patches while watching a
keynote ;)
> Perhaps we should be switching to state->rs_cxt
> while in end_heap_rewrite() also though?
I think just applying Aant's patch is fine.
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-03-26 18:33:08 | Re: [PATCH] Negative Transition Aggregate Functions (WIP) |
Previous Message | Stephen Frost | 2014-03-26 17:41:27 | Re: Missing pfree in logical_heap_rewrite_flush_mappings() |