summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane2025-04-10 18:18:07 +0000
committerTom Lane2025-04-10 18:18:07 +0000
commitf27eb0325b7b2cff3b880fa669913693849521dc (patch)
tree6e5594d30fb2735c5b34f1701bdec065efb1b486
parent4170298b6ecff7ce697b81e13d9a81e3b825798c (diff)
Remove useless check for negative result of ip_addrsize().
By inspection, ip_addrsize() can't return a negative result. (If it could, we'd have way bigger problems elsewhere.) So delete useless check in network_send(). Most C compilers are probably perfectly capable of removing this code by themselves, but it's confusing/misleading. Bug: #18889 Reported-by: Daniel Elishakov <[email protected]> Discussion: https://postgr.es/m/[email protected]
-rw-r--r--src/backend/utils/adt/network.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/backend/utils/adt/network.c b/src/backend/utils/adt/network.c
index 227980c14d3..f03fcc1147b 100644
--- a/src/backend/utils/adt/network.c
+++ b/src/backend/utils/adt/network.c
@@ -279,8 +279,6 @@ network_send(inet *addr, bool is_cidr)
pq_sendbyte(&buf, ip_bits(addr));
pq_sendbyte(&buf, is_cidr);
nb = ip_addrsize(addr);
- if (nb < 0)
- nb = 0;
pq_sendbyte(&buf, nb);
addrptr = (char *) ip_addr(addr);
for (i = 0; i < nb; i++)