]> granicus.if.org Git - postgresql/commit
Avoid query-lifetime memory leaks in XMLTABLE (bug #15321)
authorAndrew Gierth <rhodiumtoad@postgresql.org>
Mon, 13 Aug 2018 00:45:35 +0000 (01:45 +0100)
committerAndrew Gierth <rhodiumtoad@postgresql.org>
Mon, 13 Aug 2018 00:59:45 +0000 (01:59 +0100)
commit07172d5aff8f43cd6ce09f57a0b56a535d7eaf45
treed2925bdeb46287085e67d92d5d3869ae44dc512f
parent46b5e7c4b5befbf6ac86d827a3a58f1f02c7338e
Avoid query-lifetime memory leaks in XMLTABLE (bug #15321)

Multiple calls to XMLTABLE in a query (e.g. laterally applying it to a
table with an xml column, an important use-case) were leaking large
amounts of memory into the per-query context, blowing up memory usage.

Repair by reorganizing memory context usage in nodeTableFuncscan; use
the usual per-tuple context for row-by-row evaluations instead of
perValueCxt, and use the explicitly created context -- renamed from
perValueCxt to perTableCxt -- for arguments and state for each
individual table-generation operation.

Backpatch to PG10 where this code was introduced.

Original report by IRC user begriffs; analysis and patch by me.
Reviewed by Tom Lane and Pavel Stehule.

Discussion: https://postgr.es/m/153394403528.10284.7530399040974170549@wrigleys.postgresql.org
src/backend/executor/nodeTableFuncscan.c
src/include/nodes/execnodes.h