]> granicus.if.org Git - postgresql/commitdiff
Thank you for the advice. I concluded that current inet code has a
authorBruce Momjian <bruce@momjian.us>
Wed, 24 Feb 1999 03:17:05 +0000 (03:17 +0000)
committerBruce Momjian <bruce@momjian.us>
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)
                {