diff options
| author | Tom Lane | 2025-04-10 18:18:07 +0000 |
|---|---|---|
| committer | Tom Lane | 2025-04-10 18:18:07 +0000 |
| commit | f27eb0325b7b2cff3b880fa669913693849521dc (patch) | |
| tree | 6e5594d30fb2735c5b34f1701bdec065efb1b486 | |
| parent | 4170298b6ecff7ce697b81e13d9a81e3b825798c (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.c | 2 |
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++) |