Skip to content
Permalink
Browse files
Merge pull request #8670 from Leseratte10/master
Fix settings.txt parsing in case of weird line endings
  • Loading branch information
leoetlino committed Mar 15, 2020
2 parents 15acce8 + 3487e50 commit 33492e4
Showing 1 changed file with 9 additions and 1 deletion.
@@ -41,7 +41,7 @@ void SettingsHandler::SetBytes(Buffer&& buffer)

std::string SettingsHandler::GetValue(std::string_view key) const
{
constexpr char delim[] = "\r\n";
constexpr char delim[] = "\n";
std::string toFind = std::string(delim).append(key).append("=");
size_t found = decoded.find(toFind);

@@ -80,6 +80,14 @@ void SettingsHandler::Decrypt()
str++;
m_key = (m_key >> 31) | (m_key << 1);
}

// Decryption done. Now get rid of all CR in the output.
// The decoded file is supposed to contain Windows line endings
// (CR-LF), but sometimes also contains CR-LF-LF endings which
// confuse the parsing code, so let's just get rid of all CR
// line endings.

decoded.erase(std::remove(decoded.begin(), decoded.end(), '\x0d'), decoded.end());
}

void SettingsHandler::Reset()

0 comments on commit 33492e4

Please sign in to comment.