Mirrors/curl
0
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-01-18 17:21:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

os400sys: replace strcpy() with memcpy()

Browse Source 
Source and target are the same size, null-terminator is already present
in the target buffer.

Closes #20089
This commit is contained in:
Viktor Szakats
2025-12-24 01:13:19 +01:00
parent ca46112991
commit c882439d53
1 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
packages/OS400/os400sys.c
View File

@@ -705,14 +705,14 @@ char *Curl_ldap_get_dn_a(void *ld, LDAPMessage *entry)
return cp2;
QadrtConvertE2A(cp2, cp, i, i);
cp2[i] = '\0';
/* No way to allocate a buffer here, because it will be released by
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);
The destination buffer already has a null-terminator at the correct
position. Keep it outouched and copy the buffer without a terminator. */
memcpy(cp, cp2, i);
free(cp2);
return cp;
}
@@ -736,14 +736,14 @@ char *Curl_ldap_first_attribute_a(void *ld, LDAPMessage *entry,
return cp2;
QadrtConvertE2A(cp2, cp, i, i);
cp2[i] = '\0';
/* No way to allocate a buffer here, because it will be released by
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);
The destination buffer already has a null-terminator at the correct
position. Keep it outouched and copy the buffer without a terminator. */
memcpy(cp, cp2, i);
free(cp2);
return cp;
}
@@ -767,14 +767,14 @@ char *Curl_ldap_next_attribute_a(void *ld, LDAPMessage *entry,
return cp2;
QadrtConvertE2A(cp2, cp, i, i);
cp2[i] = '\0';
/* No way to allocate a buffer here, because it will be released by
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);
The destination buffer already has a null-terminator at the correct
position. Keep it outouched and copy the buffer without a terminator. */
memcpy(cp, cp2, i);
free(cp2);
return cp;
}