diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2020-12-28 11:41:25 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2020-12-28 11:41:25 -0500 |
| commit | ea80d8d9437e80de6506dbfe3765d834653312bf (patch) | |
| tree | 7f79a4671703a4e920aae9f1cb1969c2b15e359e /src | |
| parent | 643428c54b95d472c6d949c3c3c11f347e371a2b (diff) | |
| download | postgresql-ea80d8d9437e80de6506dbfe3765d834653312bf.tar.gz postgresql-ea80d8d9437e80de6506dbfe3765d834653312bf.zip | |
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
Diffstat (limited to 'src')
| -rw-r--r-- | src/pl/plpgsql/src/expected/plpgsql_call.out | 21 | ||||
| -rw-r--r-- | src/pl/plpgsql/src/pl_exec.c | 4 | ||||
| -rw-r--r-- | src/pl/plpgsql/src/sql/plpgsql_call.sql | 24 |
3 files changed, 47 insertions, 2 deletions
diff --git a/src/pl/plpgsql/src/expected/plpgsql_call.out b/src/pl/plpgsql/src/expected/plpgsql_call.out index 97385716110..c804a3ffbfe 100644 --- a/src/pl/plpgsql/src/expected/plpgsql_call.out +++ b/src/pl/plpgsql/src/expected/plpgsql_call.out @@ -152,6 +152,27 @@ CALL test_proc7(100, -1, -1); 0 | 1 (1 row) +-- inner COMMIT with output arguments +CREATE PROCEDURE test_proc7c(x int, INOUT a int, INOUT b numeric) +LANGUAGE plpgsql +AS $$ +BEGIN + a := x / 10; + b := x / 2; + COMMIT; +END; +$$; +CREATE PROCEDURE test_proc7cc(_x int) +LANGUAGE plpgsql +AS $$ +DECLARE _a int; _b numeric; +BEGIN + CALL test_proc7c(_x, _a, _b); + RAISE NOTICE '_x: %,_a: %, _b: %', _x, _a, _b; +END +$$; +CALL test_proc7cc(10); +NOTICE: _x: 10,_a: 1, _b: 5 -- named parameters and defaults CREATE PROCEDURE test_proc8a(INOUT a int, INOUT b int) LANGUAGE plpgsql diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c index 112f6ab0ae0..733ad44c74b 100644 --- a/src/pl/plpgsql/src/pl_exec.c +++ b/src/pl/plpgsql/src/pl_exec.c @@ -2235,8 +2235,8 @@ exec_stmt_call(PLpgSQL_execstate *estate, PLpgSQL_stmt_call *stmt) int i; ListCell *lc; - /* Use eval_mcontext for any cruft accumulated here */ - oldcontext = MemoryContextSwitchTo(get_eval_mcontext(estate)); + /* Use stmt_mcontext for any cruft accumulated here */ + oldcontext = MemoryContextSwitchTo(get_stmt_mcontext(estate)); /* * Get the parsed CallStmt, and look up the called procedure diff --git a/src/pl/plpgsql/src/sql/plpgsql_call.sql b/src/pl/plpgsql/src/sql/plpgsql_call.sql index d506809ddbf..c61a75be9bd 100644 --- a/src/pl/plpgsql/src/sql/plpgsql_call.sql +++ b/src/pl/plpgsql/src/sql/plpgsql_call.sql @@ -141,6 +141,30 @@ $$; CALL test_proc7(100, -1, -1); +-- inner COMMIT with output arguments + +CREATE PROCEDURE test_proc7c(x int, INOUT a int, INOUT b numeric) +LANGUAGE plpgsql +AS $$ +BEGIN + a := x / 10; + b := x / 2; + COMMIT; +END; +$$; + +CREATE PROCEDURE test_proc7cc(_x int) +LANGUAGE plpgsql +AS $$ +DECLARE _a int; _b numeric; +BEGIN + CALL test_proc7c(_x, _a, _b); + RAISE NOTICE '_x: %,_a: %, _b: %', _x, _a, _b; +END +$$; + +CALL test_proc7cc(10); + -- named parameters and defaults |