Skip to content

lib: remove the final strncpy() calls - #14830

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/no-strncpy
Closed

bagder wants to merge 2 commits into
masterfrom
bagder/no-strncpy

Conversation

@bagder

@bagder bagder commented Sep 9, 2024

Copy link
Copy Markdown
Member

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

gethostname: return failure if the buffer is too small instead
@dfandrich

Copy link
Copy Markdown
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
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.

2 participants