mirror/u-boot-2016
1
0
Fork 0
mirror of https://git.codelinaro.org/clo/qsdk/oss/boot/u-boot-2016.git synced 2026-03-02 23:44:15 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Sysupgrade: Fixing kw issues in string functions.

Browse source 
This change replaces banned function namely, strncat and 
strncpy to strlcat and strlcpy respectively.

Change-Id: Ic02f59b29d1e58a937670f5e53dd1199cb55e53e
Signed-off-by: Avinash Pandey <avinasv@codeaurora.org>
This commit is contained in:
Avinash Pandey 2018-02-20 12:10:17 +05:30 • committed by Gerrit - the friendly Code Review server
parent d8c656785c
commit 7eeddceddf
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
tools/sysupgrade.c
View file

@ -202,14 +202,14 @@ int get_sections(void)
for (i = 0, sec = &sections[0]; i < NO_OF_SECTIONS; i++, sec++) {
if (strstr(file->d_name, sec->type)) {
if (sec->pre_op) {
strncat(sec->tmp_file, file->d_name,
strlcat(sec->tmp_file, file->d_name,
sizeof(sec->tmp_file));
if (!sec->pre_op(sec)) {
printf("Error extracting kernel from ubi\n");
return 0;
}
} else {
strncat(sec->file, file->d_name,
strlcat(sec->file, file->d_name,
sizeof(sec->file));
}
if (!check_mbn_elf(&sec)) {
@ -247,7 +247,7 @@ int load_sections(void)
for (i = 0, sec = &sections[0]; i < NO_OF_SECTIONS; i++, sec++) {
if (strstr(file->d_name, sec->type)) {
if (sec->pre_op) {
strncat(sec->tmp_file, file->d_name,
strlcat(sec->tmp_file, file->d_name,
sizeof(sec->tmp_file));
if (!sec->pre_op(sec)) {
printf("Error extracting %s from ubi\n",
@ -256,7 +256,7 @@ int load_sections(void)
return 0;
}
} else {
strncat(sec->file, file->d_name,
strlcat(sec->file, file->d_name,
sizeof(sec->file));
}
sec->is_present = PRESENT;
@ -403,7 +403,7 @@ char *find_value(char *buffer, char *search, int size)
for (i = 0; i < CERT_SIZE; i++) {
for (j = 0; search[j] && (buffer[i + j] == search[j]); j++);
if (search[j] == '\0') {
strncpy(value, &buffer[i - size], size);
strlcpy(value, &buffer[i - size], size);
value[size - 1] = '\0';
return value;
}
@ -1219,7 +1219,7 @@ int generate_hash(char *cert, char *sw_file, char *hw_file)
free(oem_model_id_str);
return 0;
}
strncpy(sw_file, tmp, 32);
strlcpy(sw_file, tmp, 32);
generate_hwid_opad(hw_id_str, oem_id_str, oem_model_id_str, &hwid_xor_opad);
tmp = create_xor_ipad_opad(f_hw_xor, &hwid_xor_opad);
@ -1230,7 +1230,7 @@ int generate_hash(char *cert, char *sw_file, char *hw_file)
free(oem_model_id_str);
return 0;
}
strncpy(hw_file, tmp, 32);
strlcpy(hw_file, tmp, 32);
free(sw_id_str);
free(hw_id_str);