From ba321706578de8402d50214a9e79a65835cdd821 Mon Sep 17 00:00:00 2001 From: Ondrej Zajicek Date: Wed, 28 Apr 2010 00:39:57 +0200 Subject: [PATCH] Better support for /31 networks. --- lib/ip.c | 14 +------------- lib/ipv4.h | 3 ++- lib/ipv6.h | 4 +++- nest/neighbor.c | 8 ++++++-- proto/ospf/iface.c | 2 +- sysdep/bsd/krt-sock.c | 10 ++++++++++ sysdep/linux/netlink/netlink.c | 6 +++++- sysdep/unix/krt-iface.c | 8 +++++--- 8 files changed, 33 insertions(+), 22 deletions(-) diff --git a/lib/ip.c b/lib/ip.c index 0afe9796..791a6013 100644 --- a/lib/ip.c +++ b/lib/ip.c @@ -150,18 +150,6 @@ void ipa_ntoh(ip_addr x) { DUMMY } */ 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, int pxlen) { DUMMY } - /** * ipa_class_mask - guess netmask according to address class * @x: IP address @@ -172,7 +160,7 @@ ip_addr ipa_opposite(ip_addr x, int pxlen) { DUMMY } * routing protocols transferring no prefix lengths nor netmasks * and this function could be useful to them. */ -ip_addr ipa_classify(ip_addr x) { DUMMY } +ip_addr ipa_class_mask(ip_addr x) { DUMMY } /** * ipa_from_u32 - convert IPv4 address to an integer diff --git a/lib/ipv4.h b/lib/ipv4.h index 74f1e88f..f258d37e 100644 --- a/lib/ipv4.h +++ b/lib/ipv4.h @@ -56,7 +56,8 @@ typedef u32 ip_addr; #define ipa_ntoh(x) x = _MI(ntohl(_I(x))) #define ipa_classify(x) ipv4_classify(_I(x)) #define ipa_has_link_scope(x) ipv4_has_link_scope(_I(x)) -#define ipa_opposite(x,len) _MI(_I(x) ^ (len == 30 ? 3 : 1)) +#define ipa_opposite_m1(x) _MI(_I(x) ^ 1) +#define ipa_opposite_m2(x) _MI(_I(x) ^ 3) #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/lib/ipv6.h b/lib/ipv6.h index e15c57af..6f8e7b3c 100644 --- a/lib/ipv6.h +++ b/lib/ipv6.h @@ -62,7 +62,9 @@ typedef struct ipv6_addr { #define ipa_ntoh(x) ipv6_ntoh(&(x)) #define ipa_classify(x) ipv6_classify(&(x)) #define ipa_has_link_scope(x) ipv6_has_link_scope(&(x)) -/* ipa_opposite and ipa_class_mask don't make sense with IPv6 */ +#define ipa_opposite_m1(x) ({ ip_addr _a=(x); _MI(_I0(_a),_I1(_a),_I2(_a),_I3(_a) ^ 1); }) +#define ipa_opposite_m2(x) ({ ip_addr _a=(x); _MI(_I0(_a),_I1(_a),_I2(_a),_I3(_a) ^ 3); }) +/* ipa_class_mask don't make sense with IPv6 */ /* ipa_from_u32 and ipa_to_u32 replaced by ipa_build */ #define ipa_build(a,b,c,d) _MI(a,b,c,d) #define ipa_compare(x,y) ipv6_compare(x,y) diff --git a/nest/neighbor.c b/nest/neighbor.c index 25b98dbd..7e7cc491 100644 --- a/nest/neighbor.c +++ b/nest/neighbor.c @@ -75,9 +75,13 @@ if_connected(ip_addr *a, struct iface *i) /* -1=error, 1=match, 0=no match */ { if (ipa_in_net(*a, b->prefix, b->pxlen)) { - if (ipa_equal(*a, b->prefix) || /* Network address */ - ipa_equal(*a, b->brd)) /* Broadcast */ +#ifndef IPV6 + if ((b->pxlen < (BITS_PER_IP_ADDRESS - 1)) && + (ipa_equal(*a, b->prefix) || /* Network address */ + ipa_equal(*a, b->brd))) /* Broadcast */ return -1; +#endif + return b->scope; } } diff --git a/proto/ospf/iface.c b/proto/ospf/iface.c index b5a509a7..6faed3a5 100644 --- a/proto/ospf/iface.c +++ b/proto/ospf/iface.c @@ -361,7 +361,7 @@ ospf_iface_sm(struct ospf_iface *ifa, int event) u8 ospf_iface_clasify(struct iface *ifa, struct ifa *addr) { - if (addr->flags & IA_UNNUMBERED) + if (ipa_nonzero(addr->opposite)) return OSPF_IT_PTP; if ((ifa->flags & (IF_MULTIACCESS | IF_MULTICAST)) == diff --git a/sysdep/bsd/krt-sock.c b/sysdep/bsd/krt-sock.c index 4817df39..12ebe6de 100644 --- a/sysdep/bsd/krt-sock.c +++ b/sysdep/bsd/krt-sock.c @@ -546,7 +546,17 @@ krt_read_addr(struct ks_msg *msg) #endif if (iface->flags & IF_MULTIACCESS) + { ifa.prefix = ipa_and(ifa.ip, ipa_mkmask(masklen)); + + if (i->ifa_prefixlen == BITS_PER_IP_ADDRESS - 1) + ifa.opposite = ipa_opposite_m1(ifa.ip); + +#ifndef IPV6 + if (i->ifa_prefixlen == BITS_PER_IP_ADDRESS - 2) + ifa.opposite = ipa_opposite_m2(ifa.ip); +#endif + } else /* PtP iface */ { ifa.flags |= IA_UNNUMBERED; diff --git a/sysdep/linux/netlink/netlink.c b/sysdep/linux/netlink/netlink.c index aa0f2c45..72f06cc2 100644 --- a/sysdep/linux/netlink/netlink.c +++ b/sysdep/linux/netlink/netlink.c @@ -390,9 +390,13 @@ nl_parse_addr(struct nlmsghdr *h) ip_addr netmask = ipa_mkmask(ifa.pxlen); ifa.prefix = ipa_and(ifa.ip, netmask); ifa.brd = ipa_or(ifa.ip, ipa_not(netmask)); + if (i->ifa_prefixlen == BITS_PER_IP_ADDRESS - 1) + ifa.opposite = ipa_opposite_m1(ifa.ip); + #ifndef IPV6 if (i->ifa_prefixlen == BITS_PER_IP_ADDRESS - 2) - ifa.opposite = ipa_opposite(ifa.ip, i->ifa_prefixlen); + ifa.opposite = ipa_opposite_m2(ifa.ip); + if ((ifi->flags & IF_BROADCAST) && a[IFA_BROADCAST]) { ip_addr xbrd; diff --git a/sysdep/unix/krt-iface.c b/sysdep/unix/krt-iface.c index e0158b6b..5fda0d1d 100644 --- a/sysdep/unix/krt-iface.c +++ b/sysdep/unix/krt-iface.c @@ -89,7 +89,7 @@ scan_ifs(struct ifreq *r, int cnt) { err = "SIOCGIFNETMASK"; goto faulty; } get_sockaddr((struct sockaddr_in *) &r->ifr_addr, &netmask, NULL, 0); l = ipa_mklen(netmask); - if (l < 0 || l == 31) + if (l < 0) { log(L_ERR "%s: Invalid netmask (%x)", i.name, netmask); goto bad; @@ -139,8 +139,10 @@ scan_ifs(struct ifreq *r, int cnt) i.flags |= IF_BROADCAST; if (a.pxlen < 30) i.flags |= IF_MULTIACCESS; - else - a.opposite = ipa_opposite(a.ip, a.pxlen); + if (a.pxlen == 30) + ifa.opposite = ipa_opposite_m2(ifa.ip); + if (a.pxlen == 31) + ifa.opposite = ipa_opposite_m1(ifa.ip); } else a.brd = a.opposite;