From 057021df0d699f9c21368ab0fa51fe821cc9a544 Mon Sep 17 00:00:00 2001 From: Martin Mares Date: Mon, 25 Aug 2008 11:19:49 +0000 Subject: Fix behavior of ipa_opposite(). It was giving wrong results on /30 networks. --- lib/ip.c | 3 ++- lib/ipv4.h | 2 +- sysdep/linux/netlink/netlink.c | 2 +- sysdep/unix/krt-iface.c | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/ip.c b/lib/ip.c index b940bf5..0afe979 100644 --- a/lib/ip.c +++ b/lib/ip.c @@ -153,13 +153,14 @@ int ipa_classify(ip_addr x) { DUMMY } /** * ipa_opposite - return address of point-to-point neighbor * @x: IP address of our end of the link + * @pxlen: network prefix length * * ipa_opposite() returns an address of the opposite end of a numbered * point-to-point link. * * This function is available in IPv4 version only. */ -ip_addr ipa_opposite(ip_addr x) { DUMMY } +ip_addr ipa_opposite(ip_addr x, int pxlen) { DUMMY } /** * ipa_class_mask - guess netmask according to address class diff --git a/lib/ipv4.h b/lib/ipv4.h index 49aa13a..4c4fab9 100644 --- a/lib/ipv4.h +++ b/lib/ipv4.h @@ -53,7 +53,7 @@ typedef u32 ip_addr; #define ipa_hton(x) x = _MI(htonl(_I(x))) #define ipa_ntoh(x) x = _MI(ntohl(_I(x))) #define ipa_classify(x) ipv4_classify(_I(x)) -#define ipa_opposite(x) _MI(_I(x) ^ 1) +#define ipa_opposite(x,len) _MI(_I(x) ^ (len == 30 ? 3 : 1)) #define ipa_class_mask(x) _MI(ipv4_class_mask(_I(x))) #define ipa_from_u32(x) _MI(x) #define ipa_to_u32(x) _I(x) diff --git a/sysdep/linux/netlink/netlink.c b/sysdep/linux/netlink/netlink.c index a54e115..a70428e 100644 --- a/sysdep/linux/netlink/netlink.c +++ b/sysdep/linux/netlink/netlink.c @@ -377,7 +377,7 @@ nl_parse_addr(struct nlmsghdr *h) ifa.brd = ipa_or(ifa.ip, ipa_not(netmask)); #ifndef IPV6 if (i->ifa_prefixlen == BITS_PER_IP_ADDRESS - 2) - ifa.opposite = ipa_opposite(ifa.ip); + ifa.opposite = ipa_opposite(ifa.ip, i->ifa_prefixlen); if ((ifi->flags & IF_BROADCAST) && a[IFA_BROADCAST]) { memcpy(&xbrd, RTA_DATA(a[IFA_BROADCAST]), sizeof(xbrd)); diff --git a/sysdep/unix/krt-iface.c b/sysdep/unix/krt-iface.c index 5cc7880..37f9d0b 100644 --- a/sysdep/unix/krt-iface.c +++ b/sysdep/unix/krt-iface.c @@ -140,7 +140,7 @@ scan_ifs(struct ifreq *r, int cnt) if (a.pxlen < 30) i.flags |= IF_MULTIACCESS; else - a.opposite = ipa_opposite(a.ip); + a.opposite = ipa_opposite(a.ip, a.pxlen); } else a.brd = a.opposite; -- cgit v1.2.3