postgres.git / summary / log / commit / refs

commit    d37965965ad30aacad4d46cfa5ca42b416331ce6
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Mon Dec 28 16:55:23 2020 +0000
First released in: 14.0, 11.11, 12.6, 13.2

    Further fix thinko in plpgsql memory leak fix.
    
    There's a second call of get_eval_mcontext() that should also be
    get_stmt_mcontext().  This is actually dead code, since no
    interesting allocations happen before switching back to the
    original context, but we should keep it in sync with the other
    call to forestall possible future bugs.
    
    Discussion: https://postgr.es/m/f075f7be-c654-9aa8-3ffc-e9214622f02a@enterprisedb.com


 src/pl/plpgsql/src/pl_exec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c
index e0c49c1e266..d380e2ec61d 100644
--- a/src/pl/plpgsql/src/pl_exec.c
+++ b/src/pl/plpgsql/src/pl_exec.c
@@ -2259,7 +2259,7 @@ exec_stmt_call(PLpgSQL_execstate *estate, PLpgSQL_stmt_call *stmt)
 			row->varnos = (int *) palloc(sizeof(int) * list_length(funcargs));
 
 			if (!local_plan)
-				MemoryContextSwitchTo(get_eval_mcontext(estate));
+				MemoryContextSwitchTo(get_stmt_mcontext(estate));
 
 			/*
 			 * Examine procedure's argument list.  Each output arg position


[parent: fd1347f9d8de]