Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 00ac616

Browse files
committed
Catch stack overflow when recursing in transformFromClauseItem().
Most parts of the parser can expect that the stack overflow check in transformExprRecurse() will trigger before things get desperate. However, transformFromClauseItem() can recurse directly to self without having analyzed any expressions, so it's possible to drive it to a stack-overrun crash. Add a check to prevent that. Per bug #17583 from Egor Chindyaskin. Back-patch to all supported branches. Richard Guo Discussion: https://postgr.es/m/17583-33be55b9f981f75c@postgresql.org
1 parent 94478f4 commit 00ac616

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/backend/parser/parse_clause.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1056,6 +1056,9 @@ transformFromClauseItem(ParseState *pstate, Node *n,
10561056
ParseNamespaceItem **top_nsitem,
10571057
List **namespace)
10581058
{
1059+
/* Guard against stack overflow due to overly deep subtree */
1060+
check_stack_depth();
1061+
10591062
if (IsA(n, RangeVar))
10601063
{
10611064
/* Plain relation reference, or perhaps a CTE reference */

0 commit comments

Comments
 (0)