Skip to content

lib: remove the final strncpy() calls #14830

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

Closed
wants to merge 2 commits into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Sep 9, 2024

wolfssl: use strcpy() as the target buffer is > 40 bytes

gethostname: return failure if the buffer is too small instead

@bagder bagder added the tidy-up label Sep 9, 2024

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
wolfssl: use strcpy() as the target buffer is > 40 bytes

gethostname: return failure if the buffer is too small instead

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
make "lib/.checksrc" enable it
@dfandrich
Copy link
Contributor

Analysis of PR #14830 at a03bfd17:

Test 142 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Test 353 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Generated by Testclutch

@bagder bagder closed this in 344a177 Sep 9, 2024
bagder added a commit that referenced this pull request Sep 9, 2024

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
make "lib/.checksrc" enable it

Closes #14830
@bagder bagder deleted the bagder/no-strncpy branch September 9, 2024 11:48
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
wolfssl: use strcpy() as the target buffer is > 40 bytes

gethostname: return failure if the buffer is too small instead

Closes curl#14830
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants