Thank you for the advice. I concluded that current inet code has a
authorBruce Momjian
Wed, 24 Feb 1999 03:17:05 +0000 (03:17 +0000)
committerBruce Momjian
Wed, 24 Feb 1999 03:17:05 +0000 (03:17 +0000)
portability problem. Included patches should be applied to both
current and 6.4 tree. I have tested on LinuxPPC, FreeBSD and Solaris
2.6. Now the inet regression tests on these platforms are all happy.
---
Tatsuo Ishii

src/backend/utils/adt/network.c

index bb2d9070e6a10bd4518e613b4ae87a634395c763..fbe087f7b3f5d4391d139f6f017c7fd92aaf985b 100644 (file)
@@ -3,7 +3,7 @@
  * is for IP V4 CIDR notation, but prepared for V6: just
  * add the necessary bits where the comments indicate.
  *
- * $Id: network.c,v 1.5 1999/01/01 04:17:13 momjian Exp $
+ * $Id: network.c,v 1.6 1999/02/24 03:17:05 momjian Exp $
  * Jon Postel RIP 16 Oct 1998
  */
 
@@ -356,7 +356,12 @@ network_broadcast(inet *ip)
    if (ip_family(ip) == AF_INET)
    {
        /* It's an IP V4 address: */
-       int addr = htonl(ntohl(ip_v4addr(ip)) | (0xffffffff >> ip_bits(ip)));
+       int addr;
+       unsigned long mask = 0xffffffff;
+
+       if (ip_bits(ip) < 32)
+           mask >>= ip_bits(ip);
+       addr = htonl(ntohl(ip_v4addr(ip)) | mask);
 
        if (inet_net_ntop(AF_INET, &addr, 32, tmp, sizeof(tmp)) == NULL)
        {