commit 3ae2be69cc7d99979e00d2ea9217496b8529bdd6
author Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> Thu Sep 08 03:43:24 2022 -0700
committer GitHub <noreply@github.com> Thu Sep 08 03:43:24 2022 -0700
tree c1378d48f3b11487c27596058f4eb4aba20f5baf
parent 6ee7a6b947c607ac79e5b66177405787dcaa8c06

gh-96652: Fix faulthandler chained signal without sigaction() (GH-96666)


Fix the faulthandler implementation of faulthandler.register(signal,
chain=True) if the sigaction() function is not available: don't call
the previous signal handler if it's NULL.
(cherry picked from commit c580a81af91af4b9df85e466f8b48c3c9c86c3df)

Co-authored-by: Victor Stinner <vstinner@python.org>

Misc/NEWS.d/next/Library/2022-09-07-22-49-37.gh-issue-96652.YqOKxI.rst

diff --git a/Misc/NEWS.d/next/Library/2022-09-07-22-49-37.gh-issue-96652.YqOKxI.rst b/Misc/NEWS.d/next/Library/2022-09-07-22-49-37.gh-issue-96652.YqOKxI.rst
new file mode 100644
index 0000000..1d04db7
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2022-09-07-22-49-37.gh-issue-96652.YqOKxI.rst
@@ -0,0 +1,3 @@
+Fix the faulthandler implementation of ``faulthandler.register(signal,
+chain=True)`` if the ``sigaction()`` function is not available: don't call
+the previous signal handler if it's NULL. Patch by Victor Stinner.

Modules/faulthandler.c

diff --git a/Modules/faulthandler.c b/Modules/faulthandler.c
index 4af8702..3ae6269 100644
--- a/Modules/faulthandler.c
+++ b/Modules/faulthandler.c
@@ -877,7 +877,7 @@ faulthandler_user(int signum)
         errno = save_errno;
     }
 #else
-    if (user->chain) {
+    if (user->chain && user->previous != NULL) {
         errno = save_errno;
         /* call the previous signal handler */
         user->previous(signum);