-
Notifications
You must be signed in to change notification settings - Fork 29.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
src: set SA_SIGINFO flag when using sa_sigaction #34648
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
For `sigaction()` the `SA_SIGINFO` flag should be set when setting the handler via the `sa_sigaction` member of the `sigaction` struct.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: James M Snell <[email protected]>
So I'm not sure why but it looks like on AIX if |
FWIW I've distilled the problem down to https://github.com/richardlau/signals |
This needs a rebase |
For
sigaction()
theSA_SIGINFO
flag should be set when settingthe handler via the
sa_sigaction
member of thesigaction
struct.This unfortunately doesn't fix #34410 but AFAICT looks correct based
on the
sigaction()
docs.Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes