NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd5f767c

committed

Fix corner-case coredump in _SPI_error_callback().

I noticed that _SPI_execute_plan initially sets spierrcontext.arg = NULL,and only fills it in some time later. If an error were to happen inbetween, _SPI_error_callback would try to dereference the null pointer.This is unlikely --- there's not much between those points exceptpush-snapshot calls --- but it's clearly not impossible. Tweak thecallback to do nothing if the pointer isn't set yet.It's been like this for awhile, so back-patch to all supported branches.

1 parent70dc7ad commitd5f767cCopy full SHA for d5f767c

File tree

1 file changed

-0

lines changed

src/backend/executor
- spi.c

1 file changed

-0

lines changed

`‎src/backend/executor/spi.c`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2407,6 +2407,9 @@ _SPI_error_callback(void *arg)`
`2407`	`2407`	`constcharquery= (constchar)arg;`
`2408`	`2408`	`intsyntaxerrposition;`
`2409`	`2409`
	`2410`	`+if (query==NULL)/* in case arg wasn't set yet */`
	`2411`	`+return;`
	`2412`	`+`
`2410`	`2413`	`/*`
`2411`	`2414`	`* If there is a syntax error position, convert to internal syntax error;`
`2412`	`2415`	`* otherwise treat the query as an item of context stack`

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!

Commitd5f767c

File tree

1 file changed

1 file changed

`‎src/backend/executor/spi.c`

0 commit comments