NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit8b56928

committed

Add a stack overflow check to copyObject().

There are some code paths, such as SPI_execute(), where we invokecopyObject() on raw parse trees before doing parse analysis on them. Sincethe bison grammar is capable of building heavily nested parsetrees whileitself using only minimal stack depth, this means that copyObject() can bethe front-line function that hits stack overflow before anything else does.Accordingly, it had better have a check_stack_depth() call. I did a bit ofperformance testing and found that this slows down copyObject() by only afew percent, so the hit ought to be negligible in the context of completeprocessing of a query.Per off-list report from Toshihide Katayama. Back-patch to all supportedbranches.

1 parentaf1a614 commit8b56928Copy full SHA for 8b56928

File tree

1 file changed

-0

lines changed

src/backend/nodes
- copyfuncs.c

1 file changed

-0

lines changed

`‎src/backend/nodes/copyfuncs.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@`
`22`	`22`
`23`	`23`	`#include"postgres.h"`
`24`	`24`
	`25`	`+#include"miscadmin.h"`
`25`	`26`	`#include"nodes/plannodes.h"`
`26`	`27`	`#include"nodes/relation.h"`
`27`	`28`	`#include"utils/datum.h"`
`@@ -3667,6 +3668,9 @@ copyObject(void *from)`
`3667`	`3668`	`if (from==NULL)`
`3668`	`3669`	`returnNULL;`
`3669`	`3670`
	`3671`	`+/* Guard against stack overflow due to overly complex expressions */`
	`3672`	`+check_stack_depth();`
	`3673`	`+`
`3670`	`3674`	`switch (nodeTag(from))`
`3671`	`3675`	`{`
`3672`	`3676`	`/*`

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!

Commit8b56928

File tree

1 file changed

1 file changed

`‎src/backend/nodes/copyfuncs.c`

0 commit comments