Tom Lane wrote: > Alvaro Herrera <alvhe...@commandprompt.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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers