tree 5ff4799a978291a1049c98260ef489d75bfcc9f2
parent 8619b62655067eb5ec647eb580169f7c06ac0ffa
author Jakob Widauer <jakob.widauer@gmail.com> 1747932293 +0200
committer Copybara-Service <copybara-worker@google.com> 1747932554 -0700

[tsan] Fix nested signal handling (#138599)

This PR fixes the bug reported in #134358.

In the current implementation of the tsan posix interceptors, the signal
set does not get restored to the correct original set, if a signal
handler gets called, while already inside of a signal handler. This
leads to the wrong signal set being set for the thread in which the
signal handler was called.

To fix this I introduced a stack of `__sanitizer_sigset_t` to keep all
the correct old signal sets and restore them in the correct order.

There was also already an existing test that tested nested / recursive
signal handlers, but it was disabled.
I therefore reenabled it, made it more robust by waiting for the second
thread to have been properly started and added checks for the signal
sets.
This test then failed before the introduction of the interceptor fix and
didn't fail with the fix.

@dvyukov What are your thoughts?

GitOrigin-RevId: 1aa746d300d72042aaa48e7982f76a823aed8cb3