NotificationsYou must be signed in to change notification settings
Fork4.9k
Star17.7k

Commit42e538f

committed

Fix pg_dump's sigTermHandler() to use _exit() not exit().

sigTermHandler() tried to be careful to invoke only operations thatare safe to do in a signal handler. But for some reason we forgotthat exit(3) is not among those, because it calls atexit handlersthat might do various random things. (pg_dump itself installs noatexit handlers, but e.g. OpenSSL does.) That led to crashes orlockups when attempting to terminate a parallel dump or restorevia a signal.Fix by calling _exit() instead.Per bug #16199 from Raúl Marín. Back-patch to all supported branches.Discussion:https://postgr.es/m/16199-cb2f121146a96f9b@postgresql.org

1 parenteb9d1f0 commit42e538fCopy full SHA for 42e538f

File tree

1 file changed

-2

lines changed

src/bin/pg_dump
- parallel.c

1 file changed

-2

lines changed

`‎src/bin/pg_dump/parallel.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -574,8 +574,11 @@ sigTermHandler(SIGNAL_ARGS)`
`574`	`574`	`write_stderr("terminated by user\n");`
`575`	`575`	`}`
`576`	`576`
`577`		`-/* And die. */`
`578`		`-exit(1);`
	`577`	`+/*`
	`578`	`+ * And die, using _exit() not exit() because the latter will invoke atexit`
	`579`	`+ * handlers that can fail if we interrupted related code.`
	`580`	`+ */`
	`581`	`+_exit(1);`
`579`	`582`	`}`
`580`	`583`
`581`	`584`	`/*`

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!

Commit42e538f

File tree

1 file changed

1 file changed

`‎src/bin/pg_dump/parallel.c`

0 commit comments