]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
dnsdist: Remove a second useless check, useless static casts 13138/head
authorRemi Gacogne <remi.gacogne@powerdns.com>
Tue, 15 Aug 2023 16:07:52 +0000 (18:07 +0200)
committerPeter van Dijk <peter.van.dijk@powerdns.com>
Tue, 15 Aug 2023 16:29:57 +0000 (18:29 +0200)
pdns/dnsdistdist/dnsdist-discovery.cc

index 2d3c66e722cd11c1dd10d543b005020fcbef8c18..5bb61ba450918995cdc3526706351e17099ec80e 100644 (file)
@@ -275,7 +275,7 @@ bool ServiceDiscovery::getDiscoveredConfig(const UpgradeableBackend& upgradeable
     const struct timeval remainingTime = {.tv_sec = backend->d_config.tcpRecvTimeout, .tv_usec = 0};
     uint16_t responseSize = 0;
     auto got = readn2WithTimeout(sock.getHandle(), &responseSize, sizeof(responseSize), remainingTime);
-    if (static_cast<size_t>(got) != sizeof(responseSize)) {
+    if (got != sizeof(responseSize)) {
       if (g_verbose) {
         warnlog("Error while waiting for the ADD upgrade response size from backend %s: %d", addr.toStringWithPort(), got);
       }
@@ -285,7 +285,7 @@ bool ServiceDiscovery::getDiscoveredConfig(const UpgradeableBackend& upgradeable
     packet.resize(ntohs(responseSize));
 
     got = readn2WithTimeout(sock.getHandle(), packet.data(), packet.size(), remainingTime);
-    if (got < 0 || static_cast<size_t>(got) != packet.size()) {
+    if (got != packet.size()) {
       if (g_verbose) {
         warnlog("Error while waiting for the ADD upgrade response from backend %s: %d", addr.toStringWithPort(), got);
       }