diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2025-04-10 14:18:07 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2025-04-10 14:18:07 -0400 |
| commit | f27eb0325b7b2cff3b880fa669913693849521dc (patch) | |
| tree | 6e5594d30fb2735c5b34f1701bdec065efb1b486 /src/backend/utils/adt/network.c | |
| parent | 4170298b6ecff7ce697b81e13d9a81e3b825798c (diff) | |
| download | postgresql-f27eb0325b7b2cff3b880fa669913693849521dc.tar.gz postgresql-f27eb0325b7b2cff3b880fa669913693849521dc.zip | |
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 <dan-eli@mail.ru>
Discussion: https://postgr.es/m/18889-73d4f19e953a629e@postgresql.org
Diffstat (limited to 'src/backend/utils/adt/network.c')
| -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++) |