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

lib: remove the final strncpy() calls #14830

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.
vszakats Viktor Szakats
wolfssl: use strcpy() as the target buffer is > 40 bytes

gethostname: return failure if the buffer is too small instead
@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
@bagder bagder deleted the bagder/no-strncpy branch September 9, 2024 11:48
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