Xargon/Duckstation
1
0
Fork 0
mirror of https://github.com/RetroDECK/Duckstation.git synced 2025-01-19 14:55:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Cheats: Fix assertions tripping on unicode characters

Browse source
This commit is contained in:
Connor McLaughlin 2020-12-26 12:58:18 +10:00
parent 8debaa34d9
commit cc9c3ea41f
1 changed files with 20 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
src/core/cheats.cpp
View file

@ -147,6 +147,11 @@ static bool IsHexCharacter(char c)
return (c >= 'A' && c <= 'F') || (c >= 'a' && c <= 'f') || (c >= '0' && c <= '9'); return (c >= 'A' && c <= 'F') || (c >= 'a' && c <= 'f') || (c >= '0' && c <= '9');
} }
static int SignedCharToInt(char ch)
{
return static_cast<int>(static_cast<unsigned char>(ch));
}
static const std::string* FindKey(const KeyValuePairVector& kvp, const char* search) static const std::string* FindKey(const KeyValuePairVector& kvp, const char* search)
{ {
for (const auto& it : kvp) for (const auto& it : kvp)
@ -180,7 +185,7 @@ bool CheatList::LoadFromPCSXRString(const std::string& str)
while (std::getline(iss, line)) while (std::getline(iss, line))
{ {
char* start = line.data(); char* start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -188,7 +193,7 @@ bool CheatList::LoadFromPCSXRString(const std::string& str)
continue; continue;
char* end = start + std::strlen(start) - 1; char* end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;
@ -299,7 +304,7 @@ bool CheatList::LoadFromLibretroString(const std::string& str)
while (std::getline(iss, line)) while (std::getline(iss, line))
{ {
char* start = line.data(); char* start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -307,7 +312,7 @@ bool CheatList::LoadFromLibretroString(const std::string& str)
continue; continue;
char* end = start + std::strlen(start) - 1; char* end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;
@ -322,21 +327,21 @@ bool CheatList::LoadFromLibretroString(const std::string& str)
*equals = '\0'; *equals = '\0';
char* key_end = equals - 1; char* key_end = equals - 1;
while (key_end > start && std::isspace(*key_end)) while (key_end > start && std::isspace(SignedCharToInt(*key_end)))
{ {
*key_end = '\0'; *key_end = '\0';
key_end--; key_end--;
} }
char* value_start = equals + 1; char* value_start = equals + 1;
while (*value_start != '\0' && std::isspace(*value_start)) while (*value_start != '\0' && std::isspace(SignedCharToInt(*value_start)))
value_start++; value_start++;
if (*value_start == '\0') if (*value_start == '\0')
continue; continue;
char* value_end = value_start + std::strlen(value_start) - 1; char* value_end = value_start + std::strlen(value_start) - 1;
while (value_end > value_start && std::isspace(*value_end)) while (value_end > value_start && std::isspace(SignedCharToInt(*value_end)))
{ {
*value_end = '\0'; *value_end = '\0';
value_end--; value_end--;
@ -480,7 +485,7 @@ CheatList::Format CheatList::DetectFileFormat(const std::string& str)
while (std::getline(iss, line)) while (std::getline(iss, line))
{ {
char* start = line.data(); char* start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -488,7 +493,7 @@ CheatList::Format CheatList::DetectFileFormat(const std::string& str)
continue; continue;
char* end = start + std::strlen(start) - 1; char* end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;
@ -566,7 +571,7 @@ bool CheatList::LoadFromPackage(const std::string& game_code)
while (std::getline(iss, line)) while (std::getline(iss, line))
{ {
char* start = line.data(); char* start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -574,7 +579,7 @@ bool CheatList::LoadFromPackage(const std::string& game_code)
continue; continue;
char* end = start + std::strlen(start) - 1; char* end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;
@ -591,7 +596,7 @@ bool CheatList::LoadFromPackage(const std::string& game_code)
while (std::getline(iss, line)) while (std::getline(iss, line))
{ {
start = line.data(); start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -599,7 +604,7 @@ bool CheatList::LoadFromPackage(const std::string& game_code)
continue; continue;
end = start + std::strlen(start) - 1; end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;
@ -774,7 +779,7 @@ bool CheatCode::SetInstructionsFromString(const std::string& str)
for (std::string line; std::getline(ss, line);) for (std::string line; std::getline(ss, line);)
{ {
char* start = line.data(); char* start = line.data();
while (*start != '\0' && std::isspace(*start)) while (*start != '\0' && std::isspace(SignedCharToInt(*start)))
start++; start++;
// skip empty lines // skip empty lines
@ -782,7 +787,7 @@ bool CheatCode::SetInstructionsFromString(const std::string& str)
continue; continue;
char* end = start + std::strlen(start) - 1; char* end = start + std::strlen(start) - 1;
while (end > start && std::isspace(*end)) while (end > start && std::isspace(SignedCharToInt(*end)))
{ {
*end = '\0'; *end = '\0';
end--; end--;