diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2020-01-20 12:57:17 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2020-01-20 12:57:17 -0500 |
commit | 6f6daa1be78503ee3d171d28ed9f6843cc53d129 (patch) | |
tree | c7a5d85b2df29a95138edb276b07a462df5f1674 /src/backend/executor/nodeAgg.c | |
parent | ff0c567cbfb7aeca479f5c087d17c85ffb5b2bf7 (diff) | |
download | postgresql-6f6daa1be78503ee3d171d28ed9f6843cc53d129.tar.gz postgresql-6f6daa1be78503ee3d171d28ed9f6843cc53d129.zip |
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
Diffstat (limited to 'src/backend/executor/nodeAgg.c')
0 files changed, 0 insertions, 0 deletions