pgsql: Avoid leaking memory while evaluating arguments for a table func - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Avoid leaking memory while evaluating arguments for a table func
Date
Msg-id E1WxoLh-0002zu-KB@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/06d5eacbc0a7db22422fc07aca56f6e69b02b8ea

Modified Files
--------------
src/backend/executor/execQual.c         |   15 +++++++++++----
src/backend/executor/nodeFunctionscan.c |   15 +++++++++++++++
src/include/executor/executor.h         |    1 +
src/include/nodes/execnodes.h           |    2 ++
4 files changed, 29 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Avoid leaking memory while evaluating arguments for a table func
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Let installcheck-world pass against a server requiring a passwor