From 13fe258e91e7a92326aedf151c571994166a06d4 Mon Sep 17 00:00:00 2001 From: MeshCollider Date: Tue, 16 Oct 2018 18:19:40 +1300 Subject: [PATCH] Error if rpcpassword in conf contains a hash character --- src/util/system.cpp | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/util/system.cpp b/src/util/system.cpp index 4f5dd2d6e..f6f36c223 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -826,8 +826,10 @@ static bool GetConfigOptions(std::istream& stream, std::string& error, std::vect std::string::size_type pos; int linenr = 1; while (std::getline(stream, str)) { + bool used_hash = false; if ((pos = str.find('#')) != std::string::npos) { str = str.substr(0, pos); + used_hash = true; } const static std::string pattern = " \t\r\n"; str = TrimString(str, pattern); @@ -840,6 +842,10 @@ static bool GetConfigOptions(std::istream& stream, std::string& error, std::vect } else if ((pos = str.find('=')) != std::string::npos) { std::string name = prefix + TrimString(str.substr(0, pos), pattern); std::string value = TrimString(str.substr(pos + 1), pattern); + if (used_hash && name == "rpcpassword") { + error = strprintf("parse error on line %i, using # in rpcpassword can be ambiguous and should be avoided", linenr); + return false; + } options.emplace_back(name, value); } else { error = strprintf("parse error on line %i: %s", linenr, str);