]> granicus.if.org Git - postgresql/commit
Avoid leaking memory while evaluating arguments for a table function.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 20 Jun 2014 02:13:47 +0000 (22:13 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 20 Jun 2014 02:13:47 +0000 (22:13 -0400)
commitc1f8fb9bfb0d2a89f646b3186a004d6e4ebdecb4
tree39427db4b959016ea25b7d9afb0f4b8813cfc0a1
parent1442b426e5d724c90070f890fed36332f4b17884
Avoid leaking memory while evaluating arguments for a table function.

ExecMakeTableFunctionResult evaluated the arguments for a function-in-FROM
in the query-lifespan memory context.  This is insignificant in simple
cases where the function relation is scanned only once; but if the function
is in a sub-SELECT or is on the inside of a nested loop, any memory
consumed during argument evaluation can add up quickly.  (The potential for
trouble here had been foreseen long ago, per existing comments; but we'd
not previously seen a complaint from the field about it.)  To fix, create
an additional temporary context just for this purpose.

Per an example from MauMau.  Back-patch to all active branches.
src/backend/executor/execQual.c
src/backend/executor/nodeFunctionscan.c
src/include/executor/executor.h
src/include/nodes/execnodes.h