ndp: fixup relay
authorSteven Barth <steven@midlink.org>
Tue, 14 Jul 2015 09:05:31 +0000 (11:05 +0200)
committerSteven Barth <steven@midlink.org>
Tue, 14 Jul 2015 09:05:31 +0000 (11:05 +0200)
src/dhcpv6-ia.c
src/ndp.c
src/odhcpd.c
src/odhcpd.h

index 0bcb4530d10d41dcd112d1f502f2ff3c693bf4a8..6ea49b0a97cf8b6432dfd9ebef63e0e1471f9658 100644 (file)
@@ -363,7 +363,7 @@ static void apply_lease(struct interface *iface, struct dhcpv6_assignment *a, bo
                struct in6_addr prefix = addrs[i].addr;
                prefix.s6_addr32[1] |= htonl(a->assigned);
                odhcpd_setup_route(&prefix, (a->managed_size) ? addrs[i].prefix : a->length,
-                               iface, &a->peer.sin6_addr, add);
+                               iface, &a->peer.sin6_addr, 1024, add);
        }
 }
 
index b5bab27c1e2103d3d99014c2ec9a583a5252a8b4..d5e9a398a3f87958e72c3cbbafb2b5de937127af 100644 (file)
--- a/src/ndp.c
+++ b/src/ndp.c
@@ -37,8 +37,6 @@ static void handle_solicit(void *addr, void *data, size_t len,
                struct interface *iface, void *dest);
 static void handle_rtnetlink(void *addr, void *data, size_t len,
                struct interface *iface, void *dest);
-static ssize_t ping6(struct in6_addr *addr,
-               const struct interface *iface);
 
 static uint32_t rtnl_seqid = 0;
 static int ping_socket = -1;
@@ -209,13 +207,16 @@ int setup_ndp_interface(struct interface *iface, bool enable)
 
 // Send an ICMP-ECHO. This is less for actually pinging but for the
 // neighbor cache to be kept up-to-date.
-static ssize_t ping6(struct in6_addr *addr,
+static void ping6(struct in6_addr *addr,
                const struct interface *iface)
 {
        struct sockaddr_in6 dest = {AF_INET6, 0, 0, *addr, iface->ifindex};
        struct icmp6_hdr echo = {.icmp6_type = ICMP6_ECHO_REQUEST};
        struct iovec iov = {&echo, sizeof(echo)};
-       return odhcpd_send(ping_socket, &dest, &iov, 1, iface);
+
+       odhcpd_setup_route(addr, 128, iface, NULL, 128, true);
+       odhcpd_send(ping_socket, &dest, &iov, 1, iface);
+       odhcpd_setup_route(addr, 128, iface, NULL, 128, false);
 }
 
 
@@ -259,52 +260,6 @@ static void handle_solicit(void *addr, void *data, size_t len,
                        ping6(&req->nd_ns_target, c);
 }
 
-
-void odhcpd_setup_route(const struct in6_addr *addr, int prefixlen,
-               const struct interface *iface, const struct in6_addr *gw, bool add)
-{
-       struct req {
-               struct nlmsghdr nh;
-               struct rtmsg rtm;
-               struct rtattr rta_dst;
-               struct in6_addr dst_addr;
-               struct rtattr rta_oif;
-               uint32_t ifindex;
-               struct rtattr rta_table;
-               uint32_t table;
-               struct rtattr rta_gw;
-               struct in6_addr gw;
-       } req = {
-               {sizeof(req), 0, NLM_F_REQUEST, ++rtnl_seqid, 0},
-               {AF_INET6, prefixlen, 0, 0, 0, 0, 0, 0, 0},
-               {sizeof(struct rtattr) + sizeof(struct in6_addr), RTA_DST},
-               *addr,
-               {sizeof(struct rtattr) + sizeof(uint32_t), RTA_OIF},
-               iface->ifindex,
-               {sizeof(struct rtattr) + sizeof(uint32_t), RTA_TABLE},
-               RT_TABLE_MAIN,
-               {sizeof(struct rtattr) + sizeof(struct in6_addr), RTA_GATEWAY},
-               IN6ADDR_ANY_INIT,
-       };
-
-       if (gw)
-               req.gw = *gw;
-
-       if (add) {
-               req.nh.nlmsg_type = RTM_NEWROUTE;
-               req.nh.nlmsg_flags |= (NLM_F_CREATE | NLM_F_REPLACE);
-               req.rtm.rtm_protocol = RTPROT_STATIC;
-               req.rtm.rtm_scope = (gw) ? RT_SCOPE_UNIVERSE : RT_SCOPE_LINK;
-               req.rtm.rtm_type = RTN_UNICAST;
-       } else {
-               req.nh.nlmsg_type = RTM_DELROUTE;
-               req.rtm.rtm_scope = RT_SCOPE_NOWHERE;
-       }
-
-       req.nh.nlmsg_len = (gw) ? sizeof(req) : offsetof(struct req, rta_gw);
-       send(rtnl_event.uloop.fd, &req, req.nh.nlmsg_len, MSG_DONTWAIT);
-}
-
 // Use rtnetlink to modify kernel routes
 static void setup_route(struct in6_addr *addr, struct interface *iface, bool add)
 {
@@ -314,7 +269,7 @@ static void setup_route(struct in6_addr *addr, struct interface *iface, bool add
                        (add) ? "Learned" : "Forgot", namebuf, iface->ifname);
 
        if (iface->learn_routes)
-               odhcpd_setup_route(addr, 128, iface, NULL, add);
+               odhcpd_setup_route(addr, 128, iface, NULL, 1024, add);
 }
 
 
@@ -347,36 +302,25 @@ static void handle_rtnetlink(_unused void *addr, void *data, size_t len,
                // Inform about a change in default route
                if (is_route && rtm->rtm_dst_len == 0)
                        raise(SIGUSR1);
-               else if (is_route && rtm->rtm_dst_len == 128)
+               else if (is_route)
                        continue;
 
                // Data to retrieve
-               size_t rta_offset = (is_route) ? sizeof(*rtm) : (is_addr) ?
-                               sizeof(struct ifaddrmsg) : sizeof(*ndm);
-               uint16_t atype = (is_route) ? RTA_DST : (is_addr) ? IFA_ADDRESS : NDA_DST;
+               size_t rta_offset = (is_addr) ? sizeof(struct ifaddrmsg) : sizeof(*ndm);
+               uint16_t atype = (is_addr) ? IFA_ADDRESS : NDA_DST;
                ssize_t alen = NLMSG_PAYLOAD(nh, rta_offset);
                struct in6_addr *addr = NULL;
-               int *ifindex = (!is_route) ? &ndm->ndm_ifindex : NULL;
-               int *metric = NULL;
 
                for (struct rtattr *rta = (void*)(((uint8_t*)ndm) + rta_offset);
                                RTA_OK(rta, alen); rta = RTA_NEXT(rta, alen)) {
                        if (rta->rta_type == atype &&
                                        RTA_PAYLOAD(rta) >= sizeof(*addr)) {
                                addr = RTA_DATA(rta);
-                       } else if (is_route && rta->rta_type == RTA_OIF &&
-                                       RTA_PAYLOAD(rta) == sizeof(int)) {
-                               ifindex = (int*)RTA_DATA(rta);
-                       } else if (is_route && rta->rta_type == RTA_GATEWAY) {
-                               ifindex = NULL;
-                               break;
-                       } else if (is_route && rta->rta_type == RTA_PRIORITY) {
-                               metric = (int*)RTA_DATA(rta);
                        }
                }
 
                // Lookup interface
-               struct interface *iface = ifindex ? odhcpd_get_interface_by_index(*ifindex) : NULL;
+               struct interface *iface = odhcpd_get_interface_by_index(ndm->ndm_ifindex);
                if (!iface)
                        continue;
 
@@ -399,7 +343,7 @@ static void handle_rtnetlink(_unused void *addr, void *data, size_t len,
                                (NUD_REACHABLE | NUD_STALE | NUD_DELAY | NUD_PROBE
                                                | NUD_PERMANENT | NUD_NOARP)));
 
-               if (iface->ndp == RELAYD_RELAY && !is_route) {
+               if (iface->ndp == RELAYD_RELAY) {
                        // Replay change to all neighbor cache
                        struct {
                                struct nlmsghdr nh;
@@ -478,23 +422,6 @@ static void handle_rtnetlink(_unused void *addr, void *data, size_t len,
 
                        if (iface->dhcpv6 == RELAYD_SERVER)
                                iface->ia_reconf = true;
-               } else if (is_route) {
-                       if (iface->ndp == RELAYD_RELAY && iface->master) {
-                               // Replay on-link route changes on all slave interfaces
-                               nh->nlmsg_flags = NLM_F_REQUEST;
-
-                               if (nh->nlmsg_type == RTM_NEWROUTE)
-                                       nh->nlmsg_flags |= NLM_F_CREATE | NLM_F_REPLACE;
-
-                               struct interface *c;
-                               list_for_each_entry(c, &interfaces, head) {
-                                       if (c->ndp == RELAYD_RELAY && !c->master) {
-                                               *ifindex = c->ifindex;
-                                               *metric = (*metric & 0xffff) | (c->ifindex << 16);
-                                               send(rtnl_event.uloop.fd, nh, nh->nlmsg_len, MSG_DONTWAIT);
-                                       }
-                               }
-                       }
                }
        }
 
index 56f4498dfc8364f1b4e6593db1ed8313997c8c42..c4115424e73bd850c57b9b00eef950162372d5f0 100644 (file)
@@ -221,7 +221,7 @@ int odhcpd_iterate_interface_neighbors(const struct interface *iface,
 
                struct ndmsg *ndm = NLMSG_DATA(nhm);
                if (ndm->ndm_ifindex != iface->ifindex ||
-                               (ndm->ndm_state & NUD_FAILED))
+                               !(ndm->ndm_state & (NUD_STALE | NUD_REACHABLE | NUD_PERMANENT)))
                        continue;
 
                struct rtattr *rta = (struct rtattr*)&ndm[1];
@@ -329,6 +329,56 @@ int odhcpd_get_preferred_interface_address(int ifindex, struct in6_addr *addr)
        return ret;
 }
 
+void odhcpd_setup_route(const struct in6_addr *addr, int prefixlen,
+               const struct interface *iface, const struct in6_addr *gw,
+               int metric, bool add)
+{
+       struct req {
+               struct nlmsghdr nh;
+               struct rtmsg rtm;
+               struct rtattr rta_dst;
+               struct in6_addr dst_addr;
+               struct rtattr rta_oif;
+               uint32_t ifindex;
+               struct rtattr rta_table;
+               uint32_t table;
+               struct rtattr rta_prio;
+               uint32_t prio;
+               struct rtattr rta_gw;
+               struct in6_addr gw;
+       } req = {
+               {sizeof(req), 0, NLM_F_REQUEST, ++rtnl_seq, 0},
+               {AF_INET6, prefixlen, 0, 0, 0, 0, 0, 0, 0},
+               {sizeof(struct rtattr) + sizeof(struct in6_addr), RTA_DST},
+               *addr,
+               {sizeof(struct rtattr) + sizeof(uint32_t), RTA_OIF},
+               iface->ifindex,
+               {sizeof(struct rtattr) + sizeof(uint32_t), RTA_TABLE},
+               RT_TABLE_MAIN,
+               {sizeof(struct rtattr) + sizeof(uint32_t), RTA_PRIORITY},
+               metric,
+               {sizeof(struct rtattr) + sizeof(struct in6_addr), RTA_GATEWAY},
+               IN6ADDR_ANY_INIT,
+       };
+
+       if (gw)
+               req.gw = *gw;
+
+       if (add) {
+               req.nh.nlmsg_type = RTM_NEWROUTE;
+               req.nh.nlmsg_flags |= (NLM_F_CREATE | NLM_F_REPLACE);
+               req.rtm.rtm_protocol = RTPROT_STATIC;
+               req.rtm.rtm_scope = (gw) ? RT_SCOPE_UNIVERSE : RT_SCOPE_LINK;
+               req.rtm.rtm_type = RTN_UNICAST;
+       } else {
+               req.nh.nlmsg_type = RTM_DELROUTE;
+               req.rtm.rtm_scope = RT_SCOPE_NOWHERE;
+       }
+
+       req.nh.nlmsg_len = (gw) ? sizeof(req) : offsetof(struct req, rta_gw);
+       send(rtnl_socket, &req, req.nh.nlmsg_len, MSG_DONTWAIT);
+}
+
 struct interface* odhcpd_get_interface_by_index(int ifindex)
 {
        struct interface *iface;
index dcd2d1489b2923604aeddbdc33b06ec9ef857555..729b6291e70e5725b9e1b9496bfddeb1a3ab2fe5 100644 (file)
@@ -192,7 +192,8 @@ struct interface* odhcpd_get_interface_by_index(int ifindex);
 struct interface* odhcpd_get_master_interface(void);
 int odhcpd_urandom(void *data, size_t len);
 void odhcpd_setup_route(const struct in6_addr *addr, int prefixlen,
-               const struct interface *iface, const struct in6_addr *gw, bool add);
+               const struct interface *iface, const struct in6_addr *gw,
+               int metric, bool add);
 
 void odhcpd_run(void);
 time_t odhcpd_time(void);