Neil Conway <[EMAIL PROTECTED]> writes: > On Tue, 2008-02-26 at 00:17 -0800, Neil Conway wrote: >> You didn't comment on my proposed solution (FreeTupleDesc() iff refcount >> == -1).
I still find that entirely unsafe, particularly for something you propose to back-patch into stable branches. Negative refcount does not prove that the SRF itself hasn't still got a pointer to the tupdesc. Can't we fix it so that the tupdesc is allocated in the new special context (at least in the primary code paths), and then no explicit free is needed? regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq