diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-03-24 22:40:29 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-03-24 22:40:29 +0000 |
| commit | 8899a2aba92c4a17f422172e7c9dd0e383eefa39 (patch) | |
| tree | aea400d25b0e9c32b84004728c995cd53ab33533 /src/backend/optimizer/util/clauses.c | |
| parent | a09b9a36d3cc8e4c5cd2877b2b764dc14a78f58e (diff) | |
Replace max_expr_depth parameter with a max_stack_depth parameter that
is measured in kilobytes and checked against actual physical execution
stack depth, as per my proposal of 30-Dec. This gives us a fairly
bulletproof defense against crashing due to runaway recursive functions.
Diffstat (limited to 'src/backend/optimizer/util/clauses.c')
| -rw-r--r-- | src/backend/optimizer/util/clauses.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/backend/optimizer/util/clauses.c b/src/backend/optimizer/util/clauses.c index c006cd49a1f..b05f760ade2 100644 --- a/src/backend/optimizer/util/clauses.c +++ b/src/backend/optimizer/util/clauses.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/optimizer/util/clauses.c,v 1.166 2004/03/21 22:29:11 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/optimizer/util/clauses.c,v 1.167 2004/03/24 22:40:28 tgl Exp $ * * HISTORY * AUTHOR DATE MAJOR EVENT @@ -2347,6 +2347,10 @@ expression_tree_walker(Node *node, */ if (node == NULL) return false; + + /* Guard against stack overflow due to overly complex expressions */ + check_stack_depth(); + switch (nodeTag(node)) { case T_Var: @@ -2720,6 +2724,10 @@ expression_tree_mutator(Node *node, if (node == NULL) return NULL; + + /* Guard against stack overflow due to overly complex expressions */ + check_stack_depth(); + switch (nodeTag(node)) { case T_Var: |