Use setting->size for size when strlcpying for string variables

This commit is contained in:
twinaphex 2018-09-26 12:57:00 +02:00
parent 7ba7bc3a1d
commit 04cb02cec7
2 changed files with 15 additions and 8 deletions

View File

@ -4311,7 +4311,7 @@ static int action_ok_push_dropdown_setting_string_options_item(const char *path,
return -1;
strlcpy(setting->value.target.string, path,
sizeof(setting->value.target.string));
setting->size);
return action_cancel_pop_default(NULL, NULL, 0, 0);
}

View File

@ -779,7 +779,8 @@ static int setting_string_action_left_netplay_mitm_server(
if (!found)
offset = list_len - 1;
strlcpy(setting->value.target.string, netplay_mitm_server_list[offset].name, sizeof(setting->value.target.string));
strlcpy(setting->value.target.string,
netplay_mitm_server_list[offset].name, setting->size);
return 0;
}
@ -818,7 +819,8 @@ static int setting_string_action_right_netplay_mitm_server(
if (!found)
offset = 0;
strlcpy(setting->value.target.string, netplay_mitm_server_list[offset].name, sizeof(setting->value.target.string));
strlcpy(setting->value.target.string,
netplay_mitm_server_list[offset].name, setting->size);
return 0;
}
@ -924,7 +926,8 @@ static int setting_string_action_right_audio_device(
if (audio_device_index == (signed)ptr->size)
audio_device_index = 0;
strlcpy(setting->value.target.string, ptr->elems[audio_device_index].data, setting->size);
strlcpy(setting->value.target.string,
ptr->elems[audio_device_index].data, setting->size);
return 0;
}
@ -975,7 +978,8 @@ int setting_string_action_left_midi_input(void *data, bool wraparound)
i = (int)list->size - 1;
if (i >= 0)
{
strlcpy(setting->value.target.string, list->elems[i].data, setting->size);
strlcpy(setting->value.target.string,
list->elems[i].data, setting->size);
return 0;
}
}
@ -996,7 +1000,8 @@ int setting_string_action_right_midi_input(void *data, bool wraparound)
i = 0;
if (i >= 0 && i < (int)list->size)
{
strlcpy(setting->value.target.string, list->elems[i].data, setting->size);
strlcpy(setting->value.target.string,
list->elems[i].data, setting->size);
return 0;
}
}
@ -1017,7 +1022,8 @@ int setting_string_action_left_midi_output(void *data, bool wraparound)
i = (int)list->size - 1;
if (i >= 0)
{
strlcpy(setting->value.target.string, list->elems[i].data, setting->size);
strlcpy(setting->value.target.string,
list->elems[i].data, setting->size);
return 0;
}
}
@ -1038,7 +1044,8 @@ int setting_string_action_right_midi_output(void *data, bool wraparound)
i = 0;
if (i >= 0 && i < (int)list->size)
{
strlcpy(setting->value.target.string, list->elems[i].data, setting->size);
strlcpy(setting->value.target.string,
list->elems[i].data, setting->size);
return 0;
}
}