summaryrefslogtreecommitdiff
path: root/src/network
diff options
context:
space:
mode:
authorrubidium42 <rubidium@openttd.org>2021-04-27 20:26:56 +0200
committerrubidium42 <rubidium42@users.noreply.github.com>2021-05-13 23:13:17 +0200
commitc73d64adf984036a99d6974b130eda65dfc18c6c (patch)
tree05390f3269bf3279d51b290cde02cf17c395a626 /src/network
parentf219354f891c45dcefad2dd0a2f1fd92b07e4ba5 (diff)
downloadopenttd-c73d64adf984036a99d6974b130eda65dfc18c6c.tar.xz
Codechange: move passwords in settings to std::string
Diffstat (limited to 'src/network')
-rw-r--r--src/network/core/game_info.cpp2
-rw-r--r--src/network/network.cpp2
-rw-r--r--src/network/network_admin.cpp6
-rw-r--r--src/network/network_gui.cpp6
-rw-r--r--src/network/network_server.cpp10
5 files changed, 13 insertions, 13 deletions
diff --git a/src/network/core/game_info.cpp b/src/network/core/game_info.cpp
index 46bb42ec1..dd58ae500 100644
--- a/src/network/core/game_info.cpp
+++ b/src/network/core/game_info.cpp
@@ -127,7 +127,7 @@ void CheckGameCompatibility(NetworkGameInfo &ngi)
*/
void FillStaticNetworkServerGameInfo()
{
- _network_game_info.use_password = !StrEmpty(_settings_client.network.server_password);
+ _network_game_info.use_password = !_settings_client.network.server_password.empty();
_network_game_info.start_date = ConvertYMDToDate(_settings_game.game_creation.starting_year, 0, 1);
_network_game_info.clients_max = _settings_client.network.max_clients;
_network_game_info.companies_max = _settings_client.network.max_companies;
diff --git a/src/network/network.cpp b/src/network/network.cpp
index a87aab88c..b9160e208 100644
--- a/src/network/network.cpp
+++ b/src/network/network.cpp
@@ -887,7 +887,7 @@ bool NetworkServerStart()
if (!ServerNetworkGameSocketHandler::Listen(_settings_client.network.server_port)) return false;
/* Only listen for admins when the password isn't empty. */
- if (!StrEmpty(_settings_client.network.admin_password)) {
+ if (!_settings_client.network.admin_password.empty()) {
DEBUG(net, 5, "Starting listeners for admins");
if (!ServerNetworkAdminSocketHandler::Listen(_settings_client.network.server_admin_port)) return false;
}
diff --git a/src/network/network_admin.cpp b/src/network/network_admin.cpp
index 4e73aed0f..057ce32c0 100644
--- a/src/network/network_admin.cpp
+++ b/src/network/network_admin.cpp
@@ -84,7 +84,7 @@ ServerNetworkAdminSocketHandler::~ServerNetworkAdminSocketHandler()
*/
/* static */ bool ServerNetworkAdminSocketHandler::AllowConnection()
{
- bool accept = !StrEmpty(_settings_client.network.admin_password) && _network_admins_connected < MAX_ADMINS;
+ bool accept = !_settings_client.network.admin_password.empty() && _network_admins_connected < MAX_ADMINS;
/* We can't go over the MAX_ADMINS limit here. However, if we accept
* the connection, there has to be space in the pool. */
static_assert(NetworkAdminSocketPool::MAX_SIZE == MAX_ADMINS);
@@ -667,8 +667,8 @@ NetworkRecvStatus ServerNetworkAdminSocketHandler::Receive_ADMIN_JOIN(Packet *p)
char password[NETWORK_PASSWORD_LENGTH];
p->Recv_string(password, sizeof(password));
- if (StrEmpty(_settings_client.network.admin_password) ||
- strcmp(password, _settings_client.network.admin_password) != 0) {
+ if (_settings_client.network.admin_password.empty() ||
+ _settings_client.network.admin_password.compare(password) != 0) {
/* Password is invalid */
return this->SendError(NETWORK_ERROR_WRONG_PASSWORD);
}
diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp
index 56fdc7309..67992cac7 100644
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -1032,7 +1032,7 @@ struct NetworkStartServerWindow : public Window {
switch (widget) {
case WID_NSS_SETPWD:
/* If password is set, draw red '*' next to 'Set password' button. */
- if (!StrEmpty(_settings_client.network.server_password)) DrawString(r.right + WD_FRAMERECT_LEFT, this->width - WD_FRAMERECT_RIGHT, r.top, "*", TC_RED);
+ if (!_settings_client.network.server_password.empty()) DrawString(r.right + WD_FRAMERECT_LEFT, this->width - WD_FRAMERECT_RIGHT, r.top, "*", TC_RED);
}
}
@@ -1156,7 +1156,7 @@ struct NetworkStartServerWindow : public Window {
if (str == nullptr) return;
if (this->widget_id == WID_NSS_SETPWD) {
- strecpy(_settings_client.network.server_password, str, lastof(_settings_client.network.server_password));
+ _settings_client.network.server_password = str;
} else {
int32 value = atoi(str);
this->SetWidgetDirty(this->widget_id);
@@ -2583,7 +2583,7 @@ struct NetworkCompanyPasswordWindow : public Window {
void OnOk()
{
if (this->IsWidgetLowered(WID_NCP_SAVE_AS_DEFAULT_PASSWORD)) {
- strecpy(_settings_client.network.default_company_pass, this->password_editbox.text.buf, lastof(_settings_client.network.default_company_pass));
+ _settings_client.network.default_company_pass = this->password_editbox.text.buf;
}
NetworkChangeCompanyPassword(_local_company, this->password_editbox.text.buf);
diff --git a/src/network/network_server.cpp b/src/network/network_server.cpp
index bbadbeba4..c7a70300b 100644
--- a/src/network/network_server.cpp
+++ b/src/network/network_server.cpp
@@ -859,7 +859,7 @@ NetworkRecvStatus ServerNetworkGameSocketHandler::Receive_CLIENT_NEWGRFS_CHECKED
NetworkClientInfo *ci = this->GetInfo();
/* We now want a password from the client else we do not allow him in! */
- if (!StrEmpty(_settings_client.network.server_password)) {
+ if (!_settings_client.network.server_password.empty()) {
return this->SendNeedGamePassword();
}
@@ -957,8 +957,8 @@ NetworkRecvStatus ServerNetworkGameSocketHandler::Receive_CLIENT_GAME_PASSWORD(P
p->Recv_string(password, sizeof(password));
/* Check game password. Allow joining if we cleared the password meanwhile */
- if (!StrEmpty(_settings_client.network.server_password) &&
- strcmp(password, _settings_client.network.server_password) != 0) {
+ if (!_settings_client.network.server_password.empty() &&
+ _settings_client.network.server_password.compare(password) != 0) {
/* Password is invalid */
return this->SendError(NETWORK_ERROR_WRONG_PASSWORD);
}
@@ -1439,12 +1439,12 @@ NetworkRecvStatus ServerNetworkGameSocketHandler::Receive_CLIENT_RCON(Packet *p)
char pass[NETWORK_PASSWORD_LENGTH];
char command[NETWORK_RCONCOMMAND_LENGTH];
- if (StrEmpty(_settings_client.network.rcon_password)) return NETWORK_RECV_STATUS_OKAY;
+ if (_settings_client.network.rcon_password.empty()) return NETWORK_RECV_STATUS_OKAY;
p->Recv_string(pass, sizeof(pass));
p->Recv_string(command, sizeof(command));
- if (strcmp(pass, _settings_client.network.rcon_password) != 0) {
+ if (_settings_client.network.rcon_password.compare(pass) != 0) {
DEBUG(net, 1, "[rcon] Wrong password from client-id %d", this->client_id);
return NETWORK_RECV_STATUS_OKAY;
}