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
faulthandler prints "Segmentation fault" on non SIGSEGV signal deaths #96055
Comments
corona10
added a commit
to corona10/cpython
that referenced
this issue
Nov 6, 2022
corona10
added a commit
to corona10/cpython
that referenced
this issue
Nov 6, 2022
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Nov 7, 2022
…mber (pythongh-99162) (cherry picked from commit f626b7b) Co-authored-by: Dong-hee Na <donghee.na@python.org>
tim-one
pushed a commit
to tim-one/cpython
that referenced
this issue
Nov 7, 2022
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Nov 7, 2022
…mber (pythongh-99162) (cherry picked from commit f626b7b) Co-authored-by: Dong-hee Na <donghee.na@python.org>
miss-islington
added a commit
that referenced
this issue
Nov 7, 2022
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Nov 7, 2022
…mber (pythongh-99162) (cherry picked from commit f626b7b) Co-authored-by: Dong-hee Na <donghee.na@python.org>
miss-islington
added a commit
that referenced
this issue
Nov 7, 2022
@gpshead Can we close the issue or do we need more action? |
Good enough. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
gpshead commentedAug 17, 2022
•
edited by bedevere-bot
Bug report
The faulthandler module has a hard coded fixed small list of signals that it knows the name of. For anything not in this list it uses the final entry in its table as the value to print in the error message. This is misleading.
Ex: trigger a SIGTRAP in a Python process with faulthandler enabled.
The process exits properly indicating the correct death signal to the OS (Linux on aarch64 in this case), but the message emitted by Python is confusingly:
The cause is the
faulthandler_handlers
array of structs in https://github.com/python/cpython/blob/main/Modules/faulthandler.c#L130We should expand that to include more deadly signals. SIGTRAP led someone to notice it here. When we don't have the signal listed in the table it'd be nicer to emit an error message saying that rather than mislabelling it.
The text was updated successfully, but these errors were encountered: