]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix pg_dump's sigTermHandler() to use _exit() not exit().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 20 Jan 2020 17:57:17 +0000 (12:57 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 20 Jan 2020 17:57:17 +0000 (12:57 -0500)
commit6f6daa1be78503ee3d171d28ed9f6843cc53d129
treec7a5d85b2df29a95138edb276b07a462df5f1674
parentff0c567cbfb7aeca479f5c087d17c85ffb5b2bf7
Fix pg_dump's sigTermHandler() to use _exit() not exit().

sigTermHandler() tried to be careful to invoke only operations that
are safe to do in a signal handler.  But for some reason we forgot
that exit(3) is not among those, because it calls atexit handlers
that might do various random things.  (pg_dump itself installs no
atexit handlers, but e.g. OpenSSL does.)  That led to crashes or
lockups when attempting to terminate a parallel dump or restore
via 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
src/bin/pg_dump/parallel.c