Commite818898

tglsfdc

authored and

pull[bot]

committed

Catch stack overflow when recursing in transformFromClauseItem().

Most parts of the parser can expect that the stack overflow checkin transformExprRecurse() will trigger before things get desperate.However, transformFromClauseItem() can recurse directly to selfwithout having analyzed any expressions, so it's possible to driveit to a stack-overrun crash. Add a check to prevent that.Per bug #17583 from Egor Chindyaskin. Back-patch to all supportedbranches.Richard GuoDiscussion:https://postgr.es/m/17583-33be55b9f981f75c@postgresql.org

1 parent804407f commite818898Copy full SHA for e818898

File tree

1 file changed

-0

lines changed

src/backend/parser
- parse_clause.c

1 file changed

-0

lines changed

`‎src/backend/parser/parse_clause.c‎`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1050,6 +1050,9 @@ transformFromClauseItem(ParseState pstate, Node n,`
`1050`	`1050`	`ParseNamespaceItem**top_nsitem,`
`1051`	`1051`	`List**namespace)`
`1052`	`1052`	`{`
	`1053`	`+/* Guard against stack overflow due to overly deep subtree */`
	`1054`	`+check_stack_depth();`
	`1055`	`+`
`1053`	`1056`	`if (IsA(n,RangeVar))`
`1054`	`1057`	`{`
`1055`	`1058`	`/* Plain relation reference, or perhaps a CTE reference */`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite818898

File tree

1 file changed

1 file changed

`‎src/backend/parser/parse_clause.c‎`

0 commit comments