On Mon, May 25, 2015 at 10:17 AM, Peter Geoghegan <[email protected]> wrote: > While trying to fix a largely unrelated bug, I noticed that the new > build_tlist_index() call for the "excluded" targetlist (used by ON > CONFLICT DO UPDATE queries) does not have its memory subsequently > freed by the caller. Since every other call to build_tlist_index() > does this, and comments above build_tlist_index() encourage it, I > think the new caller should do the same. > > Attached patch adds such a pfree() call.
Yep. This looks correct to me. -- Michael -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
