Relintai/pandemonium_engine
1
0
Fork 0
mirror of https://github.com/Relintai/pandemonium_engine.git synced 2024-12-22 03:46:50 +01:00
Code Issues Packages Projects Releases Wiki Activity

Backported: Fix crash on get index for non-existing string

Browse Source 
As _buttons and _axes have both valid string and nullptr.
When iterating over them, if given key exists it will work correctly.
But if given key does not exist, it will end up with
String::operator=(nullptr). As String constructor from nullptr exists, I
use it. - Snowapril - c77b710035
This commit is contained in:
Relintai 2022-03-20 20:28:50 +01:00
parent 185cfb3291
commit a1f9614c07
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
main/input_default.cpp
View File

@ -1338,11 +1338,15 @@ String InputDefault::get_joy_button_string(int p_button) {
int InputDefault::get_joy_button_index_from_string(String p_button) {
for (int i = 0; i < JOY_BUTTON_MAX; i++) {
if (p_button == _buttons[i]) {
if (_buttons[i] == nullptr) {
break;
}
if (p_button == String(_buttons[i])) {
return i;
}
}
ERR_FAIL_V(-1);
ERR_FAIL_V_MSG(-1, vformat("Could not find a button index matching the string \"%s\".", p_button));
}
int InputDefault::get_unused_joy_id() {
@ -1361,9 +1365,13 @@ String InputDefault::get_joy_axis_string(int p_axis) {
int InputDefault::get_joy_axis_index_from_string(String p_axis) {
for (int i = 0; i < JOY_AXIS_MAX; i++) {
if (p_axis == _axes[i]) {
if (_axes[i] == nullptr) {
break;
}
if (p_axis == String(_axes[i])) {
return i;
}
}
ERR_FAIL_V(-1);
ERR_FAIL_V_MSG(-1, vformat("Could not find an axis index matching the string \"%s\".", p_axis));
}