romenskiy2012/curl
1
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-04-14 22:41:40 +03:00
Code Issues Projects Releases Packages Wiki Activity

checksrc: ban strcpy

Browse source 
No longer used in the codebase. Replacement is `curlx_strcopy()`, possibly
`memcpy()` or dynbuf.

Also:
- OS400: allow three calls.

Closes #20088
This commit is contained in:
Viktor Szakats 2025-12-24 01:08:58 +01:00
parent 532d134767
commit 8636ad55df
No known key found for this signature in database
GPG key ID: B5ABD165E2AEF201
2 changed files with 4 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

3
packages/OS400/os400sys.c
View file

@ -711,6 +711,7 @@ char *Curl_ldap_get_dn_a(void *ld, LDAPMessage *entry)
ldap_memfree() and ldap_memalloc() does not exist. The solution is to ldap_memfree() and ldap_memalloc() does not exist. The solution is to
overwrite the EBCDIC buffer with ASCII to return it. */ overwrite the EBCDIC buffer with ASCII to return it. */
/* !checksrc! disable BANNEDFUNC 1 */
strcpy(cp, cp2); strcpy(cp, cp2);
free(cp2); free(cp2);
return cp; return cp;
@ -741,6 +742,7 @@ char *Curl_ldap_first_attribute_a(void *ld, LDAPMessage *entry,
ldap_memfree() and ldap_memalloc() does not exist. The solution is to ldap_memfree() and ldap_memalloc() does not exist. The solution is to
overwrite the EBCDIC buffer with ASCII to return it. */ overwrite the EBCDIC buffer with ASCII to return it. */
/* !checksrc! disable BANNEDFUNC 1 */
strcpy(cp, cp2); strcpy(cp, cp2);
free(cp2); free(cp2);
return cp; return cp;
@ -771,6 +773,7 @@ char *Curl_ldap_next_attribute_a(void *ld, LDAPMessage *entry,
ldap_memfree() and ldap_memalloc() does not exist. The solution is to ldap_memfree() and ldap_memalloc() does not exist. The solution is to
overwrite the EBCDIC buffer with ASCII to return it. */ overwrite the EBCDIC buffer with ASCII to return it. */
/* !checksrc! disable BANNEDFUNC 1 */
strcpy(cp, cp2); strcpy(cp, cp2);
free(cp2); free(cp2);
return cp; return cp;

1
scripts/checksrc.pl
View file

@ -110,6 +110,7 @@ my %banfunc = (
"sscanf" => 1, "sscanf" => 1,
"stat" => 1, "stat" => 1,
"strcat" => 1, "strcat" => 1,
"strcpy" => 1,
"strdup" => 1, "strdup" => 1,
"strerror" => 1, "strerror" => 1,
"strncat" => 1, "strncat" => 1,