Skip to content

/cpython

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

[2.7] bpo-38730: Replace strncpy in import.c with memcpy. #17633

Merged
merged 1 commit into from Dec 17, 2019
Merged

[2.7] bpo-38730: Replace strncpy in import.c with memcpy. #17633

merged 1 commit into from Dec 17, 2019

Conversation

@benjaminp
Copy link
Contributor

benjaminp commented Dec 17, 2019 •
edited by bedevere-bot

In all these cases, we know the exact length we want copied, so memcpy is the right function to use.

https://bugs.python.org/issue38730
@benjaminp
bpo-38730: Replace strncpy in import.c with memcpy.
Loading status checks…
e76c4ce 
In all these cases, we know the exact length we want copied, so memcpy is the right function to use.
@benjaminp benjaminp added the skip news label Dec 17, 2019
@benjaminp benjaminp merged commit 052f47e into python:2.7 Dec 17, 2019
4 checks passed
4 checks passed
bedevere/issue-number Issue number 38730 found
Details
bedevere/maintenance-branch-pr Valid maintenance branch PR title.
bedevere/news "skip news" label found
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@benjaminp benjaminp deleted the benjaminp:more-strncpy-warnings branch Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA signed skip news
Projects
None yet
Milestone
No milestone
3 participants
@benjaminp @the-knights-who-say-ni @bedevere-bot
You can’t perform that action at this time.