Fix thinko in plpgsql memory leak fix.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 28 Dec 2020 16:41:25 +0000 (11:41 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 28 Dec 2020 16:41:25 +0000 (11:41 -0500)
commitea80d8d9437e80de6506dbfe3765d834653312bf
tree7f79a4671703a4e920aae9f1cb1969c2b15e359e
parent643428c54b95d472c6d949c3c3c11f347e371a2b
Fix thinko in plpgsql memory leak fix.

Commit a6b1f5365 intended to place the transient "target" list of
a CALL statement in the function's statement-lifespan context,
but I fat-fingered that and used get_eval_mcontext() instead of
get_stmt_mcontext().  The eval_mcontext belongs to the "simple
expression" infrastructure, which is destroyed at transaction end.
The net effect is that a CALL in a procedure to another procedure
that has OUT or INOUT parameters would fail if the called procedure
did a COMMIT.

Per report from Peter Eisentraut.  Back-patch to v11, like the
prior patch.

Discussion: https://postgr.es/m/f075f7be-c654-9aa8-3ffc-e9214622f02a@enterprisedb.com
src/pl/plpgsql/src/expected/plpgsql_call.out
src/pl/plpgsql/src/pl_exec.c
src/pl/plpgsql/src/sql/plpgsql_call.sql