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 | 552c310ba4ec072a2df48a08722f0af2e3703fdf (patch) | |
| tree | a3dea17fe8d91570e5b1639f90202af51d1a5cc1 /src/backend/nodes/outfuncs.c | |
| parent | 4d5cfb91156d1cc8ede245b8b363468efedf1975 (diff) | |
| download | postgresql-552c310ba4ec072a2df48a08722f0af2e3703fdf.tar.gz postgresql-552c310ba4ec072a2df48a08722f0af2e3703fdf.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/outfuncs.c')
| -rw-r--r-- | src/backend/nodes/outfuncs.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 5c03e9f2aa8..8d9280197bb 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -24,6 +24,7 @@ #include <ctype.h> #include "lib/stringinfo.h" +#include "miscadmin.h" #include "nodes/plannodes.h" #include "nodes/relation.h" #include "utils/datum.h" @@ -2941,6 +2942,9 @@ _outConstraint(StringInfo str, const Constraint *node) static void _outNode(StringInfo str, const void *obj) { + /* Guard against stack overflow due to overly complex expressions */ + check_stack_depth(); + if (obj == NULL) appendStringInfoString(str, "<>"); else if (IsA(obj, List) ||IsA(obj, IntList) || IsA(obj, OidList)) |