X Tutup
The Wayback Machine - https://web.archive.org/web/20210901110025/https://github.com/python/cpython/pull/27902
Skip to content
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

bpo-24234: fix bytes.__bytes__ to not truncate at a zero byte #27902

Merged
merged 1 commit into from Aug 23, 2021

Conversation

@mdickinson
Copy link
Member

@mdickinson mdickinson commented Aug 23, 2021

Fixes buildbot failures introduced in GH-27901.

https://bugs.python.org/issue24234

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Aug 23, 2021

🤖 New build scheduled with the buildbot fleet by @mdickinson for commit 72ea593 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

Copy link
Member

@corona10 corona10 left a comment

@mdickinson Thank you for quick fix 👍
LGTM

@mdickinson
Copy link
Member Author

@mdickinson mdickinson commented Aug 23, 2021

The s390x buildbot is reporting:

test_typing leaked [1, 1, 1] memory blocks, sum=3

@corona10
Copy link
Member

@corona10 corona10 commented Aug 23, 2021

@mdickinson Other Refleak tests doesn't report the issue :(

@mdickinson
Copy link
Member Author

@mdickinson mdickinson commented Aug 23, 2021

I'll merge anyway - this change is obviously needed. The Arch Linux buildbot failure looks unrelated, and has been happening for a while. We can keep an eye on the s390x buildbot for a while to see if the test_typing refleak is persistent.

@mdickinson mdickinson merged commit ae52591 into python:main Aug 23, 2021
70 of 74 checks passed
70 of 74 checks passed
@github-actions
Check for source changes
Details
@github-actions
Check if generated files are up to date
Details
@github-actions
Windows (x86)
Details
@github-actions
Windows (x64)
Details
@github-actions
macOS
Details
@github-actions
Ubuntu
Details
@github-actions
Ubuntu SSL tests with OpenSSL
Details
@github-actions
Address sanitizer Address sanitizer
Details
@bedevere-bot
buildbot/AMD64 Arch Linux Asan Debug PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora Refleaks PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Windows8.1 Refleaks PR Build started.
Details
@bedevere-bot
buildbot/PPC64LE RHEL7 Refleaks PR Build started.
Details
Azure Pipelines PR #20210823.13 succeeded
Details
@bedevere-bot
bedevere/issue-number Issue number 24234 found
Details
@bedevere-bot
bedevere/news "skip news" label found
@bedevere-bot
buildbot/AMD64 Arch Linux Asan PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Arch Linux TraceRefs PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Arch Linux Usan PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Debian PGO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable Clang Installed PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable Clang PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable LTO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Fedora Stable Refleaks PR Build done.
Details
@bedevere-bot
buildbot/AMD64 FreeBSD Non-Debug PR Build done.
Details
@bedevere-bot
buildbot/AMD64 FreeBSD Shared PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL7 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL7 LTO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL7 PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL7 Refleaks PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL8 FIPS Only Blake2 Builtin Hash PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL8 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL8 LTO PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL8 PR Build done.
Details
@bedevere-bot
buildbot/AMD64 RHEL8 Refleaks PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Ubuntu Shared PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Windows10 PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Windows10 Pro PR Build done.
Details
@bedevere-bot
buildbot/AMD64 Windows8.1 Non-Debug PR Build done.
Details
@bedevere-bot
buildbot/PPC64 Fedora PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable Clang Installed PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable Clang PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable LTO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE Fedora Stable Refleaks PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL7 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL7 LTO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL7 PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL8 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL8 LTO PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL8 PR Build done.
Details
@bedevere-bot
buildbot/PPC64LE RHEL8 Refleaks PR Build done.
Details
@bedevere-bot
buildbot/aarch64 Fedora Stable PR Build done.
Details
@bedevere-bot
buildbot/aarch64 RHEL8 PR Build done.
Details
@bedevere-bot
buildbot/s390x Debian PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora Clang Installed PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora Clang PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora LTO PR Build done.
Details
@bedevere-bot
buildbot/s390x Fedora PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL7 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL7 LTO PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL7 PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL7 Refleaks PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL8 LTO + PGO PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL8 LTO PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL8 PR Build done.
Details
@bedevere-bot
buildbot/s390x RHEL8 Refleaks PR Build done.
Details
@bedevere-bot
buildbot/s390x SLES PR Build done.
Details
@bedevere-bot
buildbot/x86 Gentoo Installed with X PR Build done.
Details
@bedevere-bot
buildbot/x86 Gentoo Non-Debug with X PR Build done.
Details
@bedevere-bot
buildbot/x86-64 macOS PR Build done.
Details
@mdickinson mdickinson deleted the mdickinson:bpo-24234-fix-bytes-bytes branch Aug 23, 2021
@mdickinson
Copy link
Member Author

@mdickinson mdickinson commented Aug 26, 2021

The s390x buildbot is reporting [...]

There have been 9 runs of that buildbot since; none of them have reproduced that same test_typing leak, so I think we can be reasonably confident that it wasn't caused by this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
X Tutup