dave/openttd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use std::all_of instead of std::count for checking if all 0

Browse Source
This commit is contained in:
Jonathan G Rennison
2023-06-17 17:22:59 +01:00
parent 32ec1bc77b
commit 8c600fd480
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/network/network_client.cpp
View File

@@ -415,7 +415,7 @@ NetworkRecvStatus ClientNetworkGameSocketHandler::SendKeyPasswordPacket(PacketTy
byte shared_secret[32]; // Shared secret
crypto_x25519(shared_secret, keys.x25519_priv_key, _server_x25519_pub_key);
if (std::count(shared_secret, shared_secret + 32, 0) == 32) {
if (std::all_of(shared_secret, shared_secret + 32, [](auto v) { return v == 0; })) {
/* Secret is all 0 because public key is all 0, just give up at this point */
return NETWORK_RECV_STATUS_MALFORMED_PACKET;
}

2
src/network/network_server.cpp
View File

@@ -242,7 +242,7 @@ bool ServerNetworkGameSocketHandler::ParseKeyPasswordPacket(Packet *p, NetworkSh
byte shared_secret[32]; // Shared secret
crypto_x25519(shared_secret, keys.x25519_priv_key, client_pub_key);
if (std::count(shared_secret, shared_secret + 32, 0) == 32) {
if (std::all_of(shared_secret, shared_secret + 32, [](auto v) { return v == 0; })) {
/* Secret is all 0 because public key is all 0, just reject it */
return false;
}