-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-132673: Fix ctypes.Structure
with _align_=0
#132676
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
Conversation
Misc/NEWS.d/next/Library/2025-04-18-14-34-43.gh-issue-132673.0sliCv.rst
Outdated
Show resolved
Hide resolved
…sliCv.rst Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, this is a really funny bug! LGTM, with one news nitpick.
@@ -0,0 +1,2 @@ | |||
Fix :exc:`AssertionError` raised on :class:`ctypes.Structure` with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't it a crash, not an AssertionError
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In 3.14 it was an AssertionError
.
(For 3.13, the NEWS entry should mention a crash) |
Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13. |
Sorry, @sobolevn, I could not cleanly backport this to
|
See #132673 (comment) on why I don't think that backport is needed. |
_align_ = 0
segfaults when used with empty filed list (_fields_ = []
) #132673