-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
UBsan: Remove _Py_NO_SANITIZE_UNDEFINED #133157
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
Labels
extension-modules
C modules in the Modules dir
interpreter-core
(Objects, Python, Grammar, and Parser dirs)
topic-C-API
type-bug
An unexpected behavior, bug, or error
Comments
picnixz
added a commit
that referenced
this issue
May 18, 2025
…er` (#134047) In `faulthandler_sigfpe()`, instead of using 1/0 arithmetic, we explicitly raise SIGFPE. We also remove `faulthandler._read_null()` since reading from NULL is an undefined behavior and `faulthandler` should not check for low-level C undefined behaviors.
pablogsal
pushed a commit
that referenced
this issue
Jun 10, 2025
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Jun 10, 2025
…er/pegen.c` (pythonGH-134048) (cherry picked from commit 754e7c9) Co-authored-by: Bénédikt Tran <[email protected]>
picnixz
added a commit
to picnixz/cpython
that referenced
this issue
Jun 10, 2025
…n `Parser/pegen.c` (pythonGH-134048) (cherry picked from commit 754e7c9) Co-authored-by: Bénédikt Tran <[email protected]>
picnixz
added a commit
that referenced
this issue
Jun 10, 2025
encukou
added a commit
to encukou/cpython
that referenced
this issue
Jun 10, 2025
…pat` Co-Authored-By: Bénédikt Tran <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
extension-modules
C modules in the Modules dir
interpreter-core
(Objects, Python, Grammar, and Parser dirs)
topic-C-API
type-bug
An unexpected behavior, bug, or error
Uh oh!
There was an error while loading. Please reload this page.
Bug report
Split out from the #111178 monster-issue:
We use the macro
_Py_NO_SANITIZE_UNDEFINED
to disable the UB sanitizer in some hard-to-fix cases, so that we can get a stable, regression-monitoring checker sooner.To fully fix UBsan failures, we should get rid of the macro.
At least outside test functions.
Linked PRs
_Py_NO_SANITIZE_UNDEFINED
in faulthandler #134047_Py_NO_SANITIZE_UNDEFINED
inParser/pegen.c
#134048_Py_NO_SANITIZE_UNDEFINED
inParser/pegen.c
(GH-134048) #135320Parser/pegen.c
(GH-134048) #135334_Py_NO_SANITIZE_UNDEFINED
inpyexpat
#134050_Py_NO_SANITIZE_UNDEFINED
inpyexpat
#135346The text was updated successfully, but these errors were encountered: