diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-10 11:12:43 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-10 11:12:43 -0500 |
| commit | 62999b93251eefbfcf900a0bb202721b0a422ffd (patch) | |
| tree | ac10a553f14d55e4a7756466b896d45d3b1aec3d /src/backend/nodes/nodeFuncs.c | |
| parent | a628e0c5b484eed450093258440cf858bcdb0555 (diff) | |
| download | postgresql-62999b93251eefbfcf900a0bb202721b0a422ffd.tar.gz postgresql-62999b93251eefbfcf900a0bb202721b0a422ffd.zip | |
Add stack depth checks to key recursive functions in backend/nodes/*.c.
Although copyfuncs.c has a check_stack_depth call in its recursion,
equalfuncs.c, outfuncs.c, and readfuncs.c lacked one. This seems
unwise.
Likewise fix planstate_tree_walker(), in branches where that exists.
Discussion: https://postgr.es/m/30253.1544286631@sss.pgh.pa.us
Diffstat (limited to 'src/backend/nodes/nodeFuncs.c')
| -rw-r--r-- | src/backend/nodes/nodeFuncs.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/nodes/nodeFuncs.c b/src/backend/nodes/nodeFuncs.c index a10014f755b..54b3dcff3ec 100644 --- a/src/backend/nodes/nodeFuncs.c +++ b/src/backend/nodes/nodeFuncs.c @@ -3726,6 +3726,9 @@ planstate_tree_walker(PlanState *planstate, Plan *plan = planstate->plan; ListCell *lc; + /* Guard against stack overflow due to overly complex plan trees */ + check_stack_depth(); + /* initPlan-s */ if (planstate_walk_subplans(planstate->initPlan, walker, context)) return true; |