NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite07ebd4

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 parent8ded656 commite07ebd4Copy full SHA for e07ebd4

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!

Commite07ebd4

File tree

1 file changed

1 file changed

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

0 commit comments