Tom Lane wrote:
> Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> > I think a patch to solve this is as simple as the attached.
>
> I guess I need to point out that those ereport calls already pose a far
> more substantial risk of palloc failure than the DLNewElem represents.
Hmm, do they? I mean, don't they use ErrorContext, which is supposed to
be preallocated?
> You seem to have forgotten about releasing the DLElem if the fork fails,
> too.
Doh, sorry about that.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support