romenskiy2012/curl
1
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-08-24 12:13:33 +03:00
Code Issues Projects Releases Packages Wiki Activity

curlx: curlx_strcopy() instead of strcpy()

Browse source 
This function REQUIRES the size of the target buffer as well as the
length of the source string. Meant to make it harder to do a bad
strcpy().
This commit is contained in:
Daniel Stenberg 2025-12-21 23:40:24 +01:00
parent 6b9c75e219
commit f249199426
No known key found for this signature in database
GPG key ID: 5CC908FDB71E12C2
30 changed files with 197 additions and 97 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/hostip.c
View file

@ -60,6 +60,7 @@
#include "select.h"
#include "strcase.h"
#include "easy_lock.h"
#include "curlx/strcopy.h"
#include "curlx/strparse.h"
#if defined(CURLRES_SYNCH) && \
@ -622,7 +623,7 @@ static struct Curl_addrinfo *get_localhost6(int port, const char *name)
ca->ai_addr = (void *)((char *)ca + sizeof(struct Curl_addrinfo));
memcpy(ca->ai_addr, &sa6, ss_size);
ca->ai_canonname = (char *)ca->ai_addr + ss_size;
strcpy(ca->ai_canonname, name);
curlx_strcopy(ca->ai_canonname, hostlen + 1, name, hostlen);
return ca;
}
#else
@ -659,7 +660,7 @@ static struct Curl_addrinfo *get_localhost(int port, const char *name)
ca->ai_addr = (void *)((char *)ca + sizeof(struct Curl_addrinfo));
memcpy(ca->ai_addr, &sa, ss_size);
ca->ai_canonname = (char *)ca->ai_addr + ss_size;
strcpy(ca->ai_canonname, name);
curlx_strcopy(ca->ai_canonname, hostlen + 1, name, hostlen);
ca6 = get_localhost6(port, name);
if(!ca6)