KEMBAR78
gh-111178: fix UBSan failures for `Modules/_testcapimodule.c` by picnixz · Pull Request #131614 · python/cpython · GitHub
Skip to content

gh-111178: fix UBSan failures for Modules/_testcapimodule.c #131614

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Mar 24, 2025
Merged

gh-111178: fix UBSan failures for Modules/_testcapimodule.c #131614

merged 6 commits into from
Mar 24, 2025

Conversation

@picnixz
Copy link
Member

@picnixz picnixz commented Mar 23, 2025 •
edited by bedevere-app bot
Loading

@bedevere-app bedevere-app bot added tests Tests in the Lib/test dir awaiting core review labels Mar 23, 2025
@bedevere-app bedevere-app bot mentioned this pull request Mar 23, 2025
Closed
@picnixz picnixz self-assigned this Mar 24, 2025
vstinner
vstinner approved these changes Mar 24, 2025
View reviewed changes
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@picnixz picnixz enabled auto-merge (squash) March 24, 2025 09:51
@picnixz picnixz merged commit af29d5c into python:main Mar 24, 2025
41 checks passed
@picnixz picnixz deleted the fix/ubsan/testcapimodule-111178 branch March 24, 2025 10:26
seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
@picnixz @seehwan80
pythongh-111178: fix UBSan failures for Modules/_testcapimodule.c (p…
25b7bd7 
…ython#131614)

Fix UBSan failures for various classes in `Modules/_testcapimodule.c`,
remove some redundant casts and add some `Py_UNUSED()` usages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner vstinner approved these changes

@skirpichev skirpichev skirpichev approved these changes

Assignees

@picnixz picnixz

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@picnixz @vstinner @skirpichev