]> git.puffer.fish Git - mirror/frr.git/commitdiff
bfdd: use UDP sockets instead of raw
authorRafael Zalamena <rzalamena@opensourcerouting.org>
Wed, 29 Aug 2018 18:17:49 +0000 (15:17 -0300)
committerRafael Zalamena <rzalamena@opensourcerouting.org>
Mon, 3 Sep 2018 13:41:33 +0000 (10:41 -0300)
Avoid all complexities of handling ethernet/IP/UDP headers and just use
UDP sockets: let the OS kernel handle this task.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
bfdd/bfd.h
bfdd/bfd_packet.c
bfdd/bfdd.c
bfdd/bsd.c
bfdd/linux.c

index f567bff745b1b385c7301a10ac44daad38e7c1ea..3a58a8d53cfeab25e889167cb18a1267c19bc882 100644 (file)
@@ -130,14 +130,6 @@ struct bfd_echo_pkt {
 #define BFD_GETSTATE(flags) ((flags >> 6) & 0x3)
 #define BFD_ECHO_VERSION 1
 #define BFD_ECHO_PKT_LEN sizeof(struct bfd_echo_pkt)
-#define BFD_CTRL_PKT_LEN sizeof(struct bfd_pkt)
-#define IP_HDR_LEN 20
-#define UDP_HDR_LEN 8
-#define ETH_HDR_LEN 14
-#define HEADERS_MIN_LEN (ETH_HDR_LEN + IP_HDR_LEN + UDP_HDR_LEN)
-#define BFD_ECHO_PKT_TOT_LEN                                                   \
-       ((int)(ETH_HDR_LEN + IP_HDR_LEN + UDP_HDR_LEN + BFD_ECHO_PKT_LEN))
-#define BFD_RX_BUF_LEN 160
 
 enum bfd_diagnosticis {
        BD_OK = 0,
@@ -248,14 +240,10 @@ struct bfd_session {
        int ifindex;
        uint8_t local_mac[ETHERNET_ADDRESS_LENGTH];
        uint8_t peer_mac[ETHERNET_ADDRESS_LENGTH];
-       uint16_t ip_id;
 
        /* BFD session flags */
        enum bfd_session_flags flags;
 
-       uint8_t echo_pkt[BFD_ECHO_PKT_TOT_LEN]; /* Save the Echo Packet
-                                                * which will be transmitted
-                                                */
        struct bfd_session_stats stats;
 
        struct timeval uptime;   /* last up time */
@@ -405,6 +393,7 @@ struct bfd_global {
        int bg_shop6;
        int bg_mhop6;
        int bg_echo;
+       int bg_echov6;
        struct thread *bg_ev[6];
 
        int bg_csock;
@@ -482,14 +471,14 @@ int bp_udp6_shop(void);
 int bp_udp6_mhop(void);
 int bp_peer_socket(struct bfd_peer_cfg *bpc);
 int bp_peer_socketv6(struct bfd_peer_cfg *bpc);
+int bp_echo_socket(void);
+int bp_echov6_socket(void);
 
 void ptm_bfd_snd(struct bfd_session *bfd, int fbit);
 void ptm_bfd_echo_snd(struct bfd_session *bfd);
 
 int bfd_recv_cb(struct thread *t);
 
-uint16_t checksum(uint16_t *buf, int len);
-
 
 /*
  * event.c
@@ -599,29 +588,13 @@ int ptm_bfd_notify(struct bfd_session *bs);
 /*
  * OS compatibility functions.
  */
-struct udp_psuedo_header {
-       uint32_t saddr;
-       uint32_t daddr;
-       uint8_t reserved;
-       uint8_t protocol;
-       uint16_t len;
-};
-
-#define UDP_PSUEDO_HDR_LEN sizeof(struct udp_psuedo_header)
-
 #if defined(BFD_LINUX) || defined(BFD_BSD)
 int ptm_bfd_fetch_ifindex(const char *ifname);
 void ptm_bfd_fetch_local_mac(const char *ifname, uint8_t *mac);
 void fetch_portname_from_ifindex(int ifindex, char *ifname, size_t ifnamelen);
-int ptm_bfd_echo_sock_init(void);
 #endif /* BFD_LINUX || BFD_BSD */
 
-#ifdef BFD_LINUX
-uint16_t udp4_checksum(struct iphdr *iph, uint8_t *buf, int len);
-#endif /* BFD_LINUX */
-
 #ifdef BFD_BSD
-uint16_t udp4_checksum(struct ip *ip, uint8_t *buf, int len);
 ssize_t bsd_echo_sock_read(int sd, uint8_t *buf, ssize_t *buflen,
                           struct sockaddr_storage *ss, socklen_t *sslen,
                           uint8_t *ttl, uint32_t *id);
index 455ba07e6a16b3886330e04c2d7b131e765957a3..21c1f157acb73404252c32277c421575e9f1c432 100644 (file)
 
 #include "bfd.h"
 
-/*
- * Definitions
- */
-struct bfd_raw_echo_pkt {
-#ifdef BFD_LINUX
-       struct iphdr ip;
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       struct ip ip;
-#endif /* BFD_BSD */
-       struct udphdr udp;
-       struct bfd_echo_pkt data;
-};
-
-#define IP_ECHO_PKT_LEN (IP_HDR_LEN + UDP_HDR_LEN + BFD_ECHO_PKT_LEN)
-#define UDP_ECHO_PKT_LEN (UDP_HDR_LEN + BFD_ECHO_PKT_LEN)
-
 
 /*
  * Prototypes
  */
-static uint16_t ptm_bfd_gen_IP_ID(struct bfd_session *bfd);
-static void ptm_bfd_echo_pkt_create(struct bfd_session *bfd);
-static int ptm_bfd_echo_loopback(uint8_t *pkt, int pkt_len, struct sockaddr *ss,
-                                socklen_t sslen);
 static int ptm_bfd_process_echo_pkt(int s);
+int _ptm_bfd_send(struct bfd_session *bs, uint16_t *port, const void *data,
+                 size_t datalen);
 
 static void bfd_sd_reschedule(int sd);
 ssize_t bfd_recv_ipv4(int sd, uint8_t *msgbuf, size_t msgbuflen, uint8_t *ttl,
@@ -73,6 +54,9 @@ ssize_t bfd_recv_ipv6(int sd, uint8_t *msgbuf, size_t msgbuflen, uint8_t *ttl,
                      char *port, size_t portlen, char *vrfname,
                      size_t vrfnamelen, struct sockaddr_any *local,
                      struct sockaddr_any *peer);
+int bp_udp_send(int sd, uint8_t ttl, uint8_t *data, size_t datalen,
+               struct sockaddr *to, socklen_t tolen);
+int bp_bfd_echo_in(int sd, uint8_t *ttl, uint32_t *my_discr);
 
 /* socket related prototypes */
 static void bp_set_ipopts(int sd);
@@ -84,65 +68,17 @@ static void bp_bind_ipv6(int sd, uint16_t port);
 /*
  * Functions
  */
-uint16_t checksum(uint16_t *buf, int len)
-{
-       int nbytes = len;
-       int sum = 0;
-       uint16_t csum = 0;
-       int size = sizeof(uint16_t);
-
-       while (nbytes > 1) {
-               sum += *buf++;
-               nbytes -= size;
-       }
-
-       if (nbytes == 1) {
-               *(uint8_t *)(&csum) = *(uint8_t *)buf;
-               sum += csum;
-       }
-
-       sum = (sum >> 16) + (sum & 0xFFFF);
-       sum += (sum >> 16);
-       csum = ~sum;
-       return csum;
-}
-
-static uint16_t ptm_bfd_gen_IP_ID(struct bfd_session *bfd)
-{
-       return (++bfd->ip_id);
-}
-
-static int _ptm_bfd_send(struct bfd_session *bs, bool use_layer2,
-                        uint16_t *port, const void *data, size_t datalen)
+int _ptm_bfd_send(struct bfd_session *bs, uint16_t *port, const void *data,
+                 size_t datalen)
 {
        struct sockaddr *sa;
        struct sockaddr_in sin;
        struct sockaddr_in6 sin6;
-#ifdef BFD_LINUX
-       struct sockaddr_ll dll;
-#endif /* BFD_LINUX */
        socklen_t slen;
        ssize_t rv;
        int sd = -1;
 
-       if (use_layer2) {
-#ifdef BFD_LINUX
-               memset(&dll, 0, sizeof(dll));
-               dll.sll_family = AF_PACKET;
-               dll.sll_protocol = htons(ETH_P_IP);
-               memcpy(dll.sll_addr, bs->peer_mac, ETHERNET_ADDRESS_LENGTH);
-               dll.sll_halen = htons(ETHERNET_ADDRESS_LENGTH);
-               dll.sll_ifindex = bs->ifindex;
-
-               sd = bglobal.bg_echo;
-               sa = (struct sockaddr *)&dll;
-               slen = sizeof(dll);
-#else
-               /* TODO: implement layer 2 send for *BSDs. */
-               log_warning("packet-send: not implemented");
-               return -1;
-#endif
-       } else if (BFD_CHECK_FLAG(bs->flags, BFD_SESS_FLAG_IPV6)) {
+       if (BFD_CHECK_FLAG(bs->flags, BFD_SESS_FLAG_IPV6)) {
                memset(&sin6, 0, sizeof(sin6));
                sin6.sin6_family = AF_INET6;
                sin6.sin6_addr = bs->shop.peer.sa_sin6.sin6_addr;
@@ -184,252 +120,63 @@ static int _ptm_bfd_send(struct bfd_session *bs, bool use_layer2,
        return 0;
 }
 
-static void ptm_bfd_echo_pkt_create(struct bfd_session *bfd)
-{
-       struct bfd_raw_echo_pkt ep;
-       uint8_t *pkt = bfd->echo_pkt;
-
-       memset(&ep, 0, sizeof(ep));
-       memset(bfd->echo_pkt, 0, sizeof(bfd->echo_pkt));
-
-       /* Construct ethernet header information */
-       memcpy(pkt, bfd->peer_mac, ETHERNET_ADDRESS_LENGTH);
-       pkt = pkt + ETHERNET_ADDRESS_LENGTH;
-       memcpy(pkt, bfd->local_mac, ETHERNET_ADDRESS_LENGTH);
-       pkt = pkt + ETHERNET_ADDRESS_LENGTH;
-#ifdef BFD_LINUX
-       pkt[0] = ETH_P_IP / 256;
-       pkt[1] = ETH_P_IP % 256;
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       pkt[0] = ETHERTYPE_IP / 256;
-       pkt[1] = ETHERTYPE_IP % 256;
-#endif /* BFD_BSD */
-       pkt += 2;
-
-       /* Construct IP header information */
-#ifdef BFD_LINUX
-       ep.ip.version = 4;
-       ep.ip.ihl = 5;
-       ep.ip.tos = 0;
-       ep.ip.tot_len = htons(IP_ECHO_PKT_LEN);
-       ep.ip.id = htons(ptm_bfd_gen_IP_ID(bfd));
-       ep.ip.frag_off = 0;
-       ep.ip.ttl = BFD_TTL_VAL;
-       ep.ip.protocol = IPPROTO_UDP;
-       ep.ip.saddr = bfd->local_ip.sa_sin.sin_addr.s_addr;
-       ep.ip.daddr = bfd->shop.peer.sa_sin.sin_addr.s_addr;
-       ep.ip.check = checksum((uint16_t *)&ep.ip, IP_HDR_LEN);
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       ep.ip.ip_v = 4;
-       ep.ip.ip_hl = 5;
-       ep.ip.ip_tos = 0;
-       ep.ip.ip_len = htons(IP_ECHO_PKT_LEN);
-       ep.ip.ip_id = htons(ptm_bfd_gen_IP_ID(bfd));
-       ep.ip.ip_off = 0;
-       ep.ip.ip_ttl = BFD_TTL_VAL;
-       ep.ip.ip_p = IPPROTO_UDP;
-       ep.ip.ip_src = bfd->local_ip.sa_sin.sin_addr;
-       ep.ip.ip_dst = bfd->shop.peer.sa_sin.sin_addr;
-       ep.ip.ip_sum = checksum((uint16_t *)&ep.ip, IP_HDR_LEN);
-#endif /* BFD_BSD */
-
-       /* Construct UDP header information */
-#ifdef BFD_LINUX
-       ep.udp.source = htons(BFD_DEF_ECHO_PORT);
-       ep.udp.dest = htons(BFD_DEF_ECHO_PORT);
-       ep.udp.len = htons(UDP_ECHO_PKT_LEN);
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       ep.udp.uh_sport = htons(BFD_DEF_ECHO_PORT);
-       ep.udp.uh_dport = htons(BFD_DEF_ECHO_PORT);
-       ep.udp.uh_ulen = htons(UDP_ECHO_PKT_LEN);
-#endif /* BFD_BSD */
-
-       /* Construct Echo packet information */
-       ep.data.ver = BFD_ECHO_VERSION;
-       ep.data.len = BFD_ECHO_PKT_LEN;
-       ep.data.my_discr = htonl(bfd->discrs.my_discr);
-#ifdef BFD_LINUX
-       ep.udp.check =
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       ep.udp.uh_sum =
-#endif /* BFD_BSD */
-               udp4_checksum(&ep.ip, (uint8_t *)&ep.udp,
-                             UDP_ECHO_PKT_LEN);
-
-       memcpy(pkt, &ep, sizeof(ep));
-}
-
 void ptm_bfd_echo_snd(struct bfd_session *bfd)
 {
-       struct bfd_raw_echo_pkt *ep;
-       bool use_layer2 = false;
-       const void *pkt;
-       size_t pktlen;
-       uint16_t port = htons(BFD_DEF_ECHO_PORT);
+       struct sockaddr_any *sa;
+       socklen_t salen;
+       int sd;
+       struct bfd_echo_pkt bep;
+       struct sockaddr_in sin;
+       struct sockaddr_in6 sin6;
 
-       if (!BFD_CHECK_FLAG(bfd->flags, BFD_SESS_FLAG_ECHO_ACTIVE)) {
-               ptm_bfd_echo_pkt_create(bfd);
+       if (!BFD_CHECK_FLAG(bfd->flags, BFD_SESS_FLAG_ECHO_ACTIVE))
                BFD_SET_FLAG(bfd->flags, BFD_SESS_FLAG_ECHO_ACTIVE);
-       } else {
-               /* just update the checksum and ip Id */
-               ep = (struct bfd_raw_echo_pkt *)(bfd->echo_pkt + ETH_HDR_LEN);
-#ifdef BFD_LINUX
-               ep->ip.id = htons(ptm_bfd_gen_IP_ID(bfd));
-               ep->ip.check = 0;
-               ep->ip.check = checksum((uint16_t *)&ep->ip, IP_HDR_LEN);
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-               ep->ip.ip_id = htons(ptm_bfd_gen_IP_ID(bfd));
-               ep->ip.ip_sum = 0;
-               ep->ip.ip_sum = checksum((uint16_t *)&ep->ip, IP_HDR_LEN);
-#endif /* BFD_BSD */
-       }
 
-       if (use_layer2) {
-               pkt = bfd->echo_pkt;
-               pktlen = BFD_ECHO_PKT_TOT_LEN;
+       memset(&bep, 0, sizeof(bep));
+       bep.ver = BFD_ECHO_VERSION;
+       bep.len = BFD_ECHO_PKT_LEN;
+       bep.my_discr = htonl(bfd->discrs.my_discr);
+
+       sa = BFD_CHECK_FLAG(bfd->flags, BFD_SESS_FLAG_MH) ? &bfd->mhop.peer
+                                                         : &bfd->shop.peer;
+       if (BFD_CHECK_FLAG(bfd->flags, BFD_SESS_FLAG_IPV6)) {
+               sd = bglobal.bg_echov6;
+               sin6 = sa->sa_sin6;
+               sin6.sin6_port = htons(BFD_DEF_ECHO_PORT);
+#ifdef HAVE_STRUCT_SOCKADDR_SA_LEN
+               sin6.sin6_len = sizeof(sin6);
+#endif /* HAVE_STRUCT_SOCKADDR_SA_LEN */
+
+               sa = (struct sockaddr_any *)&sin6;
+               salen = sizeof(sin6);
        } else {
-               pkt = &bfd->echo_pkt[ETH_HDR_LEN + IP_HDR_LEN + UDP_HDR_LEN];
-               pktlen = BFD_ECHO_PKT_TOT_LEN
-                        - (ETH_HDR_LEN + IP_HDR_LEN + UDP_HDR_LEN);
-       }
+               sd = bglobal.bg_echo;
+               sin = sa->sa_sin;
+               sin.sin_port = htons(BFD_DEF_ECHO_PORT);
+#ifdef HAVE_STRUCT_SOCKADDR_SA_LEN
+               sin.sin_len = sizeof(sin);
+#endif /* HAVE_STRUCT_SOCKADDR_SA_LEN */
 
-       if (_ptm_bfd_send(bfd, use_layer2, &port, pkt, pktlen) != 0) {
-               log_debug("echo-packet: send failure: %s", strerror(errno));
-               return;
+               sa = (struct sockaddr_any *)&sin;
+               salen = sizeof(sin);
        }
+       if (bp_udp_send(sd, BFD_TTL_VAL, (uint8_t *)&bep, sizeof(bep),
+                       (struct sockaddr *)sa, salen)
+           == -1)
+               return;
 
        bfd->stats.tx_echo_pkt++;
 }
 
-static int ptm_bfd_echo_loopback(uint8_t *pkt, int pkt_len, struct sockaddr *ss,
-                                socklen_t sslen)
-{
-#ifdef BFD_LINUX
-       struct bfd_raw_echo_pkt *ep =
-               (struct bfd_raw_echo_pkt *)(pkt + ETH_HDR_LEN);
-       uint8_t temp_mac[ETHERNET_ADDRESS_LENGTH];
-       uint32_t temp_ip;
-       struct ethhdr *eth = (struct ethhdr *)pkt;
-
-       /* swap the mac addresses */
-       memcpy(temp_mac, eth->h_source, ETHERNET_ADDRESS_LENGTH);
-       memcpy(eth->h_source, eth->h_dest, ETHERNET_ADDRESS_LENGTH);
-       memcpy(eth->h_dest, temp_mac, ETHERNET_ADDRESS_LENGTH);
-
-       /* swap ip addresses */
-       temp_ip = ep->ip.saddr;
-       ep->ip.saddr = ep->ip.daddr;
-       ep->ip.daddr = temp_ip;
-
-       ep->ip.ttl = ep->ip.ttl - 1;
-       ep->ip.check = 0;
-       ep->ip.check = checksum((uint16_t *)ep, IP_HDR_LEN);
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD_FILTER
-       struct bfd_raw_echo_pkt_t *ep =
-               (struct bfd_raw_echo_pkt *)(pkt + ETH_HDR_LEN);
-       uint8_t temp_mac[ETHERNET_ADDRESS_LENGTH];
-       struct in_addr temp_ip;
-       struct ether_header *ether = (struct ether_header *)pkt;
-
-       /*
-        * TODO: this is not yet implemented and requires BPF code for
-        * OmniOS, NetBSD and FreeBSD9.
-        */
-
-       /* swap the mac addresses */
-       memcpy(temp_mac, ether->ether_shost, ETHERNET_ADDRESS_LENGTH);
-       memcpy(ether->ether_shost, ether->ether_dhost, ETHERNET_ADDRESS_LENGTH);
-       memcpy(ether->ether_dhost, temp_mac, ETHERNET_ADDRESS_LENGTH);
-
-       /* swap ip addresses */
-       temp_ip = ep->ip.ip_src;
-       ep->ip.ip_src = ep->ip.ip_dst;
-       ep->ip.ip_dst = temp_ip;
-
-       ep->ip.ip_ttl = ep->ip.ip_ttl - 1;
-       ep->ip.ip_sum = 0;
-       ep->ip.ip_sum = checksum((uint16_t *)ep, IP_HDR_LEN);
-#endif /* BFD_BSD_FILTER */
-
-       if (sendto(bglobal.bg_echo, pkt, pkt_len, 0, ss, sslen) < 0) {
-               log_debug("echo-loopback: send failure: %s", strerror(errno));
-               return -1;
-       }
-
-       return 0;
-}
-
 static int ptm_bfd_process_echo_pkt(int s)
 {
-       uint32_t my_discr = 0;
-       struct sockaddr_storage ss;
-       socklen_t sslen = sizeof(ss);
-       uint8_t rx_pkt[BFD_RX_BUF_LEN];
-       ssize_t pkt_len = sizeof(rx_pkt);
        struct bfd_session *bfd;
-#ifdef BFD_LINUX
-       struct bfd_raw_echo_pkt *ep;
-
-       /*
-        * valgrind: memset() ss so valgrind doesn't complain about
-        * uninitialized memory.
-        */
-       memset(&ss, 0, sizeof(ss));
-       pkt_len = recvfrom(s, rx_pkt, sizeof(rx_pkt), MSG_DONTWAIT,
-                          (struct sockaddr *)&ss, &sslen);
-       if (pkt_len <= 0) {
-               if (errno != EAGAIN)
-                       log_error("echo-packet: read failure: %s",
-                                 strerror(errno));
-
-               return -1;
-       }
-
-       /* Check if we have at least the basic headers to send back. */
-       if (pkt_len < BFD_ECHO_PKT_TOT_LEN) {
-               log_debug("echo-packet: too short (got %ld, expected %d)",
-                         pkt_len, BFD_ECHO_PKT_TOT_LEN);
-               return -1;
-       }
-
-       ep = (struct bfd_raw_echo_pkt *)(rx_pkt + ETH_HDR_LEN);
-       /* if TTL = 255, assume that the received echo packet has
-        * to be looped back
-        */
-       if (ep->ip.ttl == BFD_TTL_VAL)
-               return ptm_bfd_echo_loopback(rx_pkt, pkt_len,
-                                            (struct sockaddr *)&ss,
-                                            sizeof(struct sockaddr_ll));
-
-       my_discr = ntohl(ep->data.my_discr);
-       if (ep->data.my_discr == 0) {
-               log_debug("echo-packet: 'my discriminator' is zero");
-               return -1;
-       }
-#endif /* BFD_LINUX */
-#ifdef BFD_BSD
-       int rv;
-       uint8_t ttl;
-
-       /*
-        * bsd_echo_sock_read() already treats invalid TTL values and
-        * zeroed discriminators.
-        */
-       rv = bsd_echo_sock_read(s, rx_pkt, &pkt_len, &ss, &sslen, &ttl,
-                               &my_discr);
-       if (rv == -1)
-               return -1;
+       uint32_t my_discr = 0;
+       uint8_t ttl = 0;
 
-       if (ttl == BFD_TTL_VAL)
-               return ptm_bfd_echo_loopback(rx_pkt, pkt_len,
-                                            (struct sockaddr *)&ss, sslen);
-#endif /* BFD_BSD */
+       /* Receive and parse echo packet. */
+       if (bp_bfd_echo_in(s, &ttl, &my_discr) == -1)
+               return 0;
 
        /* Your discriminator not zero - use it to find session */
        bfd = bfd_id_lookup(my_discr);
@@ -482,7 +229,7 @@ void ptm_bfd_snd(struct bfd_session *bfd, int fbit)
        }
        cp.timers.required_min_echo = htonl(bfd->timers.required_min_echo);
 
-       if (_ptm_bfd_send(bfd, false, NULL, &cp, BFD_PKT_LEN) != 0)
+       if (_ptm_bfd_send(bfd, NULL, &cp, BFD_PKT_LEN) != 0)
                return;
 
        bfd->stats.tx_ctrl_pkt++;
@@ -692,6 +439,10 @@ static void bfd_sd_reschedule(int sd)
                bglobal.bg_ev[4] = NULL;
                thread_add_read(master, bfd_recv_cb, NULL, bglobal.bg_echo,
                                &bglobal.bg_ev[4]);
+       } else if (sd == bglobal.bg_echov6) {
+               bglobal.bg_ev[5] = NULL;
+               thread_add_read(master, bfd_recv_cb, NULL, bglobal.bg_echov6,
+                               &bglobal.bg_ev[5]);
        }
 }
 
@@ -748,7 +499,7 @@ int bfd_recv_cb(struct thread *t)
        bfd_sd_reschedule(sd);
 
        /* Handle echo packets. */
-       if (sd == bglobal.bg_echo) {
+       if (sd == bglobal.bg_echo || sd == bglobal.bg_echov6) {
                ptm_bfd_process_echo_pkt(sd);
                return 0;
        }
@@ -984,6 +735,121 @@ int bfd_recv_cb(struct thread *t)
        return 0;
 }
 
+/*
+ * bp_bfd_echo_in: proccesses an BFD echo packet. On TTL == BFD_TTL_VAL
+ * the packet is looped back or returns the my discriminator ID along
+ * with the TTL.
+ *
+ * Returns -1 on error or loopback or 0 on success.
+ */
+int bp_bfd_echo_in(int sd, uint8_t *ttl, uint32_t *my_discr)
+{
+       struct bfd_echo_pkt *bep;
+       ssize_t rlen;
+       struct sockaddr_any local, peer;
+       char port[MAXNAMELEN + 1], vrfname[MAXNAMELEN + 1];
+       uint8_t msgbuf[1516];
+
+       if (sd == bglobal.bg_echo)
+               rlen = bfd_recv_ipv4(sd, msgbuf, sizeof(msgbuf), ttl, port,
+                                    sizeof(port), vrfname, sizeof(vrfname),
+                                    &local, &peer);
+       else
+               rlen = bfd_recv_ipv6(sd, msgbuf, sizeof(msgbuf), ttl, port,
+                                    sizeof(port), vrfname, sizeof(vrfname),
+                                    &local, &peer);
+
+       /* Short packet, better not risk reading it. */
+       if (rlen < (ssize_t)sizeof(*bep)) {
+               cp_debug(false, &peer, &local, port, vrfname,
+                        "small echo packet");
+               return -1;
+       }
+
+       /* Test for loopback. */
+       if (*ttl == BFD_TTL_VAL) {
+               bp_udp_send(sd, *ttl - 1, msgbuf, rlen,
+                           (struct sockaddr *)&peer,
+                           (sd == bglobal.bg_echo) ? sizeof(peer.sa_sin)
+                                                   : sizeof(peer.sa_sin6));
+               return -1;
+       }
+
+       /* Read my discriminator from BFD Echo packet. */
+       bep = (struct bfd_echo_pkt *)msgbuf;
+       *my_discr = ntohl(bep->my_discr);
+       if (*my_discr == 0) {
+               cp_debug(false, &peer, &local, port, vrfname,
+                        "invalid echo packet discriminator (zero)");
+               return -1;
+       }
+
+       return 0;
+}
+
+int bp_udp_send(int sd, uint8_t ttl, uint8_t *data, size_t datalen,
+               struct sockaddr *to, socklen_t tolen)
+{
+       struct cmsghdr *cmsg;
+       ssize_t wlen;
+       int ttlval = ttl;
+       bool is_ipv6 = to->sa_family == AF_INET6;
+       struct msghdr msg;
+       struct iovec iov[1];
+       uint8_t msgctl[255];
+
+       /* Prepare message data. */
+       iov[0].iov_base = data;
+       iov[0].iov_len = datalen;
+
+       memset(&msg, 0, sizeof(msg));
+       memset(msgctl, 0, sizeof(msgctl));
+       msg.msg_name = to;
+       msg.msg_namelen = tolen;
+       msg.msg_iov = iov;
+       msg.msg_iovlen = 1;
+
+       /* Prepare the packet TTL information. */
+       if (ttl > 0) {
+               /* Use ancillary data. */
+               msg.msg_control = msgctl;
+               msg.msg_controllen = CMSG_LEN(sizeof(ttlval));
+
+               /* Configure the ancillary data. */
+               cmsg = CMSG_FIRSTHDR(&msg);
+               cmsg->cmsg_len = CMSG_LEN(sizeof(ttlval));
+               if (is_ipv6) {
+                       cmsg->cmsg_level = IPPROTO_IPV6;
+                       cmsg->cmsg_type = IPV6_HOPLIMIT;
+               } else {
+#if BFD_LINUX
+                       cmsg->cmsg_level = IPPROTO_IP;
+                       cmsg->cmsg_type = IP_TTL;
+#else
+                       /* FreeBSD does not support TTL in ancillary data. */
+                       msg.msg_control = NULL;
+                       msg.msg_controllen = 0;
+
+                       bp_set_ttl(sd, ttl);
+#endif /* BFD_BSD */
+               }
+               memcpy(CMSG_DATA(cmsg), &ttlval, sizeof(ttlval));
+       }
+
+       /* Send echo back. */
+       wlen = sendmsg(sd, &msg, 0);
+       if (wlen <= 0) {
+               log_debug("udp-send: loopback failure: (%d) %s", errno, strerror(errno));
+               return -1;
+       } else if (wlen < (ssize_t)datalen) {
+               log_debug("udp-send: partial send: %ld expected %ld", wlen,
+                         datalen);
+               return -1;
+       }
+
+       return 0;
+}
+
 
 /*
  * Sockets creation.
@@ -1325,3 +1191,31 @@ int bp_udp6_mhop(void)
 
        return sd;
 }
+
+int bp_echo_socket(void)
+{
+       int s;
+
+       s = socket(AF_INET, SOCK_DGRAM, 0);
+       if (s == -1)
+               log_fatal("echo-socket: socket: %s", strerror(errno));
+
+       bp_set_ipopts(s);
+       bp_bind_ip(s, BFD_DEF_ECHO_PORT);
+
+       return s;
+}
+
+int bp_echov6_socket(void)
+{
+       int s;
+
+       s = socket(AF_INET6, SOCK_DGRAM, 0);
+       if (s == -1)
+               log_fatal("echov6-socket: socket: %s", strerror(errno));
+
+       bp_set_ipv6opts(s);
+       bp_bind_ipv6(s, BFD_DEF_ECHO_PORT);
+
+       return s;
+}
index 9c75e103e77c13a86c61f06a83150c14326f8b78..250f8d21c0d30aaf5ed77b3d0871fa627d746b3b 100644 (file)
@@ -158,7 +158,8 @@ static void bg_init(void)
        bglobal.bg_mhop = bp_udp_mhop();
        bglobal.bg_shop6 = bp_udp6_shop();
        bglobal.bg_mhop6 = bp_udp6_mhop();
-       bglobal.bg_echo = ptm_bfd_echo_sock_init();
+       bglobal.bg_echo = bp_echo_socket();
+       bglobal.bg_echov6 = bp_echov6_socket();
 }
 
 int main(int argc, char *argv[])
@@ -219,6 +220,8 @@ int main(int argc, char *argv[])
                        &bglobal.bg_ev[3]);
        thread_add_read(master, bfd_recv_cb, NULL, bglobal.bg_echo,
                        &bglobal.bg_ev[4]);
+       thread_add_read(master, bfd_recv_cb, NULL, bglobal.bg_echov6,
+                       &bglobal.bg_ev[5]);
        thread_add_read(master, control_accept, NULL, bglobal.bg_csock,
                        &bglobal.bg_csockev);
 
index 632353c7071c449a7cf75e1623f520f63e8c7a3b..e0fb340e30e3087bd6fdb935b2bd42380f6d0f93 100644 (file)
 
 #include "bfd.h"
 
-/*
- * Prototypes
- */
-static const char *sockaddr_to_string(const void *sv, char *buf, size_t buflen);
-
 /*
  * Definitions.
  */
-static const char *sockaddr_to_string(const void *sv, char *buf, size_t buflen)
-{
-       const struct sockaddr *sa = sv;
-       const struct sockaddr_in *sin = sv;
-       const struct sockaddr_in6 *sin6 = sv;
-       int unknown = 1;
-
-       switch (sa->sa_family) {
-       case AF_INET:
-               if (inet_ntop(AF_INET, &sin->sin_addr, buf, buflen) != NULL)
-                       unknown = 0;
-               break;
-
-       case AF_INET6:
-               if (inet_ntop(AF_INET6, &sin6->sin6_addr, buf, buflen) != NULL)
-                       unknown = 0;
-               break;
-       }
-       if (unknown == 0)
-               return buf;
-
-       snprintf(buf, buflen, "unknown (af=%d)", sa->sa_family);
-       return buf;
-}
-
 int ptm_bfd_fetch_ifindex(const char *ifname)
 {
        return if_nametoindex(ifname);
@@ -129,127 +99,6 @@ void fetch_portname_from_ifindex(int ifindex, char *ifname, size_t ifnamelen)
                            __LINE__);
 }
 
-int ptm_bfd_echo_sock_init(void)
-{
-       int s, ttl, yes = 1;
-       struct sockaddr_in sin;
-
-       s = socket(AF_INET, SOCK_DGRAM, PF_UNSPEC);
-       if (s == -1) {
-               log_error("echo-socket: creation failed: %s", strerror(errno));
-               return -1;
-       }
-
-       memset(&sin, 0, sizeof(sin));
-#ifdef HAVE_STRUCT_SOCKADDR_SA_LEN
-       /* OmniOS doesn't have this field, but uses this code. */
-       sin.sin_len = sizeof(sin);
-#endif /* HAVE_STRUCT_SOCKADDR_SA_LEN */
-       sin.sin_family = AF_INET;
-       sin.sin_port = htons(3785);
-       if (bind(s, (struct sockaddr *)&sin, sizeof(sin)) == -1) {
-               log_error("echo-socket: bind failure: %s", strerror(errno));
-               close(s);
-               return -1;
-       }
-
-       if (setsockopt(s, IPPROTO_IP, IP_RECVTTL, &yes, sizeof(yes)) == -1) {
-               log_error("echo-socket: setsockopt(IP_RECVTTL): %s",
-                         strerror(errno));
-               close(s);
-               return -1;
-       }
-
-       ttl = BFD_TTL_VAL;
-       if (setsockopt(s, IPPROTO_IP, IP_TTL, &ttl, sizeof(ttl)) == -1) {
-               log_error("echo-socket: setsockopt(IP_TTL): %s",
-                         strerror(errno));
-               close(s);
-               return -1;
-       }
-
-       return s;
-}
-
-ssize_t bsd_echo_sock_read(int sd, uint8_t *buf, ssize_t *buflen,
-                          struct sockaddr_storage *ss, socklen_t *sslen,
-                          uint8_t *ttl, uint32_t *id)
-{
-       struct cmsghdr *cmsg;
-       struct bfd_echo_pkt *bep;
-       ssize_t readlen;
-       struct iovec iov;
-       struct msghdr msg;
-       uint8_t msgctl[255];
-       char errbuf[255];
-
-       /* Prepare socket read. */
-       memset(ss, 0, sizeof(*ss));
-       memset(&msg, 0, sizeof(msg));
-       iov.iov_base = buf;
-       iov.iov_len = *buflen;
-       msg.msg_iov = &iov;
-       msg.msg_iovlen = 1;
-       msg.msg_control = msgctl;
-       msg.msg_controllen = sizeof(msgctl);
-       msg.msg_name = ss;
-       msg.msg_namelen = *sslen;
-
-       /* Read the socket and treat errors. */
-       readlen = recvmsg(sd, &msg, 0);
-       if (readlen == 0) {
-               log_error("%s: recvmsg: socket closed", __func__);
-               return -1;
-       }
-       if (readlen == -1) {
-               if (errno == EAGAIN || errno == EWOULDBLOCK || errno == EINTR)
-                       return -1;
-
-               log_error("%s: recvmsg: (%d) %s", __func__, errno,
-                         strerror(errno));
-               return -1;
-       }
-       /* Short packet, better not risk reading it. */
-       if (readlen < (ssize_t)sizeof(*bep)) {
-               log_warning("%s: short packet (%ld of %d) from %s", __func__,
-                           readlen, sizeof(*bep),
-                           sockaddr_to_string(ss, errbuf, sizeof(errbuf)));
-               return -1;
-       }
-       *buflen = readlen;
-
-       /* Read TTL information. */
-       *ttl = 0;
-       for (cmsg = CMSG_FIRSTHDR(&msg); cmsg != NULL;
-            cmsg = CMSG_NXTHDR(&msg, cmsg)) {
-               if (cmsg->cmsg_level != IPPROTO_IP)
-                       continue;
-               if (cmsg->cmsg_type != IP_RECVTTL)
-                       continue;
-
-               *ttl = *(uint8_t *)CMSG_DATA(cmsg);
-               break;
-       }
-       if (*ttl == 0) {
-               log_debug("%s: failed to read TTL", __func__);
-               return -1;
-       }
-
-       /* Read my discriminator from BFD Echo packet. */
-       bep = (struct bfd_echo_pkt *)buf;
-       *id = bep->my_discr;
-       if (*id == 0) {
-               log_debug("%s: invalid packet discriminator from: %s", __func__,
-                         sockaddr_to_string(ss, errbuf, sizeof(errbuf)));
-               return -1;
-       }
-
-       /* Set the returned sockaddr new length. */
-       *sslen = msg.msg_namelen;
-
-       return 0;
-}
-
 int bp_bind_dev(int sd, const char *dev)
 {
        /*
@@ -260,25 +109,4 @@ int bp_bind_dev(int sd, const char *dev)
        return 0;
 }
 
-uint16_t udp4_checksum(struct ip *ip, uint8_t *buf, int len)
-{
-       char *ptr;
-       struct udp_psuedo_header pudp_hdr;
-       uint16_t csum;
-
-       pudp_hdr.saddr = ip->ip_src.s_addr;
-       pudp_hdr.daddr = ip->ip_dst.s_addr;
-       pudp_hdr.reserved = 0;
-       pudp_hdr.protocol = ip->ip_p;
-       pudp_hdr.len = htons(len);
-
-       ptr = XMALLOC(MTYPE_BFDD_TMP, UDP_PSUEDO_HDR_LEN + len);
-       memcpy(ptr, &pudp_hdr, UDP_PSUEDO_HDR_LEN);
-       memcpy(ptr + UDP_PSUEDO_HDR_LEN, buf, len);
-
-       csum = checksum((uint16_t *)ptr, UDP_PSUEDO_HDR_LEN + len);
-       XFREE(MTYPE_BFDD_TMP, ptr);
-       return csum;
-}
-
 #endif /* BFD_BSD */
index 442958ead9c5fdf1b9fe60aeccfbb2f80fa168a8..e260851ddba6a229573406c7540c3e20b3a99ec0 100644 (file)
 
 #ifdef BFD_LINUX
 
-/* XXX: fix compilation error on Ubuntu 16.04 or older. */
-#ifndef _UAPI_IPV6_H
-#define _UAPI_IPV6_H
-#endif /* _UAPI_IPV6_H */
-
-#include <linux/filter.h>
-#include <linux/if_packet.h>
-
-#include <netinet/if_ether.h>
-
-#include <net/if.h>
-#include <sys/ioctl.h>
-
 #include "bfd.h"
 
-/* Berkeley Packet filter code to filter out BFD Echo packets.
- * tcpdump -dd "(udp dst port 3785)"
- */
-static struct sock_filter bfd_echo_filter[] = {
-       {0x28, 0, 0, 0x0000000c}, {0x15, 0, 4, 0x000086dd},
-       {0x30, 0, 0, 0x00000014}, {0x15, 0, 11, 0x00000011},
-       {0x28, 0, 0, 0x00000038}, {0x15, 8, 9, 0x00000ec9},
-       {0x15, 0, 8, 0x00000800}, {0x30, 0, 0, 0x00000017},
-       {0x15, 0, 6, 0x00000011}, {0x28, 0, 0, 0x00000014},
-       {0x45, 4, 0, 0x00001fff}, {0xb1, 0, 0, 0x0000000e},
-       {0x48, 0, 0, 0x00000010}, {0x15, 0, 1, 0x00000ec9},
-       {0x6, 0, 0, 0x0000ffff},  {0x6, 0, 0, 0x00000000},
-};
-
 
 /*
  * Definitions.
@@ -114,30 +87,6 @@ void fetch_portname_from_ifindex(int ifindex, char *ifname, size_t ifnamelen)
                          ifr.ifr_name, ifname);
 }
 
-int ptm_bfd_echo_sock_init(void)
-{
-       int s;
-       struct sock_fprog bpf = {.len = sizeof(bfd_echo_filter)
-                                       / sizeof(bfd_echo_filter[0]),
-                                .filter = bfd_echo_filter};
-
-       s = socket(PF_PACKET, SOCK_RAW, htons(ETH_P_IP));
-       if (s == -1) {
-               log_error("echo-socket: creation failure: %s", strerror(errno));
-               return -1;
-       }
-
-       if (setsockopt(s, SOL_SOCKET, SO_ATTACH_FILTER, &bpf, sizeof(bpf))
-           == -1) {
-               log_error("echo-socket: setsockopt(SO_ATTACH_FILTER): %s",
-                         strerror(errno));
-               close(s);
-               return -1;
-       }
-
-       return s;
-}
-
 int bp_bind_dev(int sd __attribute__((__unused__)),
                const char *dev __attribute__((__unused__)))
 {
@@ -158,25 +107,4 @@ int bp_bind_dev(int sd __attribute__((__unused__)),
        return 0;
 }
 
-uint16_t udp4_checksum(struct iphdr *iph, uint8_t *buf, int len)
-{
-       char *ptr;
-       struct udp_psuedo_header pudp_hdr;
-       uint16_t csum;
-
-       pudp_hdr.saddr = iph->saddr;
-       pudp_hdr.daddr = iph->daddr;
-       pudp_hdr.reserved = 0;
-       pudp_hdr.protocol = iph->protocol;
-       pudp_hdr.len = htons(len);
-
-       ptr = XMALLOC(MTYPE_BFDD_TMP, UDP_PSUEDO_HDR_LEN + len);
-       memcpy(ptr, &pudp_hdr, UDP_PSUEDO_HDR_LEN);
-       memcpy(ptr + UDP_PSUEDO_HDR_LEN, buf, len);
-
-       csum = checksum((uint16_t *)ptr, UDP_PSUEDO_HDR_LEN + len);
-       XFREE(MTYPE_BFDD_TMP, ptr);
-       return csum;
-}
-
 #endif /* BFD_LINUX */