Use signal(sig, SIG_DFL); raise(sig); instead of abort() (#6518)
authorGereon Kremer <nafur42@gmail.com>
Wed, 12 May 2021 06:22:52 +0000 (08:22 +0200)
committerGitHub <noreply@github.com>
Wed, 12 May 2021 06:22:52 +0000 (08:22 +0200)
commitf41d71cbf1946a5ee5a63c062a23c7426ec5e87d
treef59260e3502307c08168afdfca370f7308cfba0b
parentcd1f1c3f308b67fc4b8f006196e5bc1f366cc10d
Use signal(sig, SIG_DFL); raise(sig); instead of abort() (#6518)

As discussed in #6484 (and https://www.gnu.org/software/libc/manual/html_node/Termination-in-Handler.html), simply calling std::abort at the end of a custom signal handler seems to be bad practice. As suggested, this PR changes these calls to instead first reset to the default signal handler for the given signal, and then calling it.
Fixes #6484.
src/main/signal_handlers.cpp