]> git.puffer.fish Git - mirror/frr.git/commitdiff
*: Replace IPV6_MAX_PREFIXLEN to IPV6_MAX_BITLEN
authorDonatas Abraitis <donatas.abraitis@gmail.com>
Thu, 1 Jul 2021 14:39:04 +0000 (17:39 +0300)
committerDonatas Abraitis <donatas.abraitis@gmail.com>
Thu, 1 Jul 2021 14:41:09 +0000 (17:41 +0300)
Just drop IPV6_MAX_PREFIXLEN at all, no need keeping both.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
16 files changed:
bgpd/bgp_network.c
ldpd/labelmapping.c
lib/prefix.h
lib/table.c
lib/zclient.c
ospf6d/ospf6_asbr.c
pimd/pim_tlv.c
pimd/pim_zlookup.c
zebra/kernel_socket.c
zebra/rtadv.c
zebra/zapi_msg.c
zebra/zebra_dplane.c
zebra/zebra_mpls.c
zebra/zebra_nhg.c
zebra/zebra_pbr.c
zebra/zebra_rib.c

index a51816116e530a3ce2e592b1fe819c3d223b3968..1ddf1f4cf729529267673986bfba91e184de5412 100644 (file)
@@ -102,8 +102,7 @@ static int bgp_md5_set_socket(int socket, union sockunion *su,
 
        /* For addresses, use the non-extended signature functionality */
        if ((su2.sa.sa_family == AF_INET && prefixlen == IPV4_MAX_PREFIXLEN)
-           || (su2.sa.sa_family == AF_INET6
-               && prefixlen == IPV6_MAX_PREFIXLEN))
+           || (su2.sa.sa_family == AF_INET6 && prefixlen == IPV6_MAX_BITLEN))
                ret = sockopt_tcp_signature(socket, &su2, password);
        else
                ret = sockopt_tcp_signature_ext(socket, &su2, prefixlen,
@@ -164,8 +163,8 @@ static int bgp_md5_set_password(struct peer *peer, const char *password)
                            peer->su.sa.sa_family) {
                                uint16_t prefixlen =
                                        peer->su.sa.sa_family == AF_INET
-                                       ? IPV4_MAX_PREFIXLEN
-                                       : IPV6_MAX_PREFIXLEN;
+                                               ? IPV4_MAX_PREFIXLEN
+                                               : IPV6_MAX_BITLEN;
 
                                /*
                                 * if we have stored a BGP vrf instance in the
@@ -746,7 +745,7 @@ int bgp_connect(struct peer *peer)
        if (peer->password) {
                uint16_t prefixlen = peer->su.sa.sa_family == AF_INET
                                             ? IPV4_MAX_PREFIXLEN
-                                            : IPV6_MAX_PREFIXLEN;
+                                            : IPV6_MAX_BITLEN;
 
                bgp_md5_set_connect(peer->fd, &peer->su, prefixlen,
                                    peer->password);
index cee9d527e859a95dd27d8ab3de21638c297cdb02..53f3c00fbb3d1d4e5aa939593f1098d0fe8ce821 100644 (file)
@@ -726,7 +726,7 @@ tlv_decode_fec_elm(struct nbr *nbr, struct ldp_msg *msg, char *buf,
                if ((map->fec.prefix.af == AF_IPV4
                     && map->fec.prefix.prefixlen > IPV4_MAX_PREFIXLEN)
                    || (map->fec.prefix.af == AF_IPV6
-                       && map->fec.prefix.prefixlen > IPV6_MAX_PREFIXLEN)) {
+                       && map->fec.prefix.prefixlen > IPV6_MAX_BITLEN)) {
                        session_shutdown(nbr, S_BAD_TLV_VAL, msg->id,
                            msg->type);
                        return (-1);
index c47e701dfd879245881d7663d68fb070910d3a3a..890dcd7efa30a344989713a1dacbbb0fa7fe05fe 100644 (file)
@@ -398,7 +398,6 @@ static inline void ipv4_addr_copy(struct in_addr *dst,
 /* Max bit/byte length of IPv6 address. */
 #define IPV6_MAX_BYTELEN    16
 #define IPV6_MAX_BITLEN    128
-#define IPV6_MAX_PREFIXLEN 128
 #define IPV6_ADDR_CMP(D,S)   memcmp ((D), (S), IPV6_MAX_BYTELEN)
 #define IPV6_ADDR_SAME(D,S)  (memcmp ((D), (S), IPV6_MAX_BYTELEN) == 0)
 #define IPV6_ADDR_COPY(D,S)  memcpy ((D), (S), IPV6_MAX_BYTELEN)
index dfd92c6189f58c47ba2ebc1a22ac7b775978642f..171b3a89708502870544c7f521ae60770be48310 100644 (file)
@@ -243,7 +243,7 @@ struct route_node *route_node_match_ipv6(struct route_table *table,
 
        memset(&p, 0, sizeof(struct prefix_ipv6));
        p.family = AF_INET6;
-       p.prefixlen = IPV6_MAX_PREFIXLEN;
+       p.prefixlen = IPV6_MAX_BITLEN;
        p.prefix = *addr;
 
        return route_node_match(table, &p);
index ffae1332af0b71e76142573005374b68b1e5fd2f..c0efeebeb325d5f196d89516ee7363665f9162ca 100644 (file)
@@ -1441,7 +1441,7 @@ int zapi_route_decode(struct stream *s, struct zapi_route *api)
                }
                break;
        case AF_INET6:
-               if (api->prefix.prefixlen > IPV6_MAX_PREFIXLEN) {
+               if (api->prefix.prefixlen > IPV6_MAX_BITLEN) {
                        flog_err(
                                EC_LIB_ZAPI_ENCODE,
                                "%s: v6 prefixlen is %d which should not be more than 128",
@@ -1460,7 +1460,7 @@ int zapi_route_decode(struct stream *s, struct zapi_route *api)
        if (CHECK_FLAG(api->message, ZAPI_MESSAGE_SRCPFX)) {
                api->src_prefix.family = AF_INET6;
                STREAM_GETC(s, api->src_prefix.prefixlen);
-               if (api->src_prefix.prefixlen > IPV6_MAX_PREFIXLEN) {
+               if (api->src_prefix.prefixlen > IPV6_MAX_BITLEN) {
                        flog_err(
                                EC_LIB_ZAPI_ENCODE,
                                "%s: SRC Prefix prefixlen received: %d is too large",
index 315449ae9a3e6f9c7588a1794f8cb854651d7140..b53fc717a69babefbea4e92e1e0c8daadc218816 100644 (file)
@@ -556,7 +556,7 @@ void ospf6_asbr_lsa_add(struct ospf6_lsa *lsa)
                         + OSPF6_PREFIX_SPACE(external->prefix.prefix_length);
                memset(&fwd_addr, 0, sizeof(struct prefix));
                fwd_addr.family = AF_INET6;
-               fwd_addr.prefixlen = IPV6_MAX_PREFIXLEN;
+               fwd_addr.prefixlen = IPV6_MAX_BITLEN;
                memcpy(&fwd_addr.u.prefix6, (caddr_t)external + offset,
                       sizeof(struct in6_addr));
 
index f21c369b8d84f20fe5516ee99d154a33a4aef507..8d6d3a416b6763e2f596c3a59e768dda7105e4df 100644 (file)
@@ -495,7 +495,7 @@ int pim_parse_addr_ucast(struct prefix *p, const uint8_t *buf, int buf_size)
                }
 
                p->family = AF_INET6;
-               p->prefixlen = IPV6_MAX_PREFIXLEN;
+               p->prefixlen = IPV6_MAX_BITLEN;
                memcpy(&p->u.prefix6, addr, sizeof(struct in6_addr));
                addr += sizeof(struct in6_addr);
 
index 9ccf1fedd421171d2c44e31deccc61bceaf0c40b..dce936b8a99ba9d4cf9ba1b54c5d4ea6f800d4d5 100644 (file)
@@ -276,7 +276,7 @@ static int zclient_read_nexthop(struct pim_instance *pim,
                        nexthop_tab[num_ifindex].ifindex = stream_getl(s);
 
                        p.family = AF_INET6;
-                       p.prefixlen = IPV6_MAX_PREFIXLEN;
+                       p.prefixlen = IPV6_MAX_BITLEN;
                        memcpy(&p.u.prefix6,
                               &nexthop_tab[num_ifindex].nexthop_addr.u.prefix6,
                               sizeof(struct in6_addr));
index 03884a916820e2d200649029b93f453d772c86e3..d3e5f42338445c54152c7019f1488c504fc8dad9 100644 (file)
@@ -1079,7 +1079,7 @@ void rtm_read(struct rt_msghdr *rtm)
                p.family = AF_INET6;
                p.u.prefix6 = dest.sin6.sin6_addr;
                if (flags & RTF_HOST)
-                       p.prefixlen = IPV6_MAX_PREFIXLEN;
+                       p.prefixlen = IPV6_MAX_BITLEN;
                else
                        p.prefixlen = ip6_masklen(mask.sin6.sin6_addr);
 
index 8cb0223a46331218a31b47c99640889d1c2f227f..976beefab05fa0dc894c0f04cd78ba1a30a2affb 100644 (file)
@@ -693,7 +693,7 @@ static void rtadv_process_advert(uint8_t *msg, unsigned int len,
        /* Create entry for neighbor if not known. */
        p.family = AF_INET6;
        IPV6_ADDR_COPY(&p.u.prefix6, &addr->sin6_addr);
-       p.prefixlen = IPV6_MAX_PREFIXLEN;
+       p.prefixlen = IPV6_MAX_BITLEN;
 
        if (!nbr_connected_check(ifp, &p))
                nbr_connected_add_ipv6(ifp, &addr->sin6_addr);
index d5969ab9bb62b78029e78fd959d7ba3d3bb5a8c9..8f2aa2fb09e596bf7cba50db4894fef067dc43a7 100644 (file)
@@ -480,7 +480,7 @@ void nbr_connected_add_ipv6(struct interface *ifp, struct in6_addr *address)
 
        p.family = AF_INET6;
        IPV6_ADDR_COPY(&p.u.prefix6, address);
-       p.prefixlen = IPV6_MAX_PREFIXLEN;
+       p.prefixlen = IPV6_MAX_BITLEN;
 
        ifc = listnode_head(ifp->nbr_connected);
        if (!ifc) {
@@ -505,7 +505,7 @@ void nbr_connected_delete_ipv6(struct interface *ifp, struct in6_addr *address)
 
        p.family = AF_INET6;
        IPV6_ADDR_COPY(&p.u.prefix6, address);
-       p.prefixlen = IPV6_MAX_PREFIXLEN;
+       p.prefixlen = IPV6_MAX_BITLEN;
 
        ifc = nbr_connected_check(ifp, &p);
        if (!ifc)
index a5d672987d3382177ed3c504f95aa629257f9e3d..90009cf17e720c739e3437f06ee28d8d1713a5d7 100644 (file)
@@ -2537,8 +2537,8 @@ static int dplane_ctx_pw_init(struct zebra_dplane_ctx *ctx,
         */
        memcpy(&p.u, &pw->nexthop, sizeof(pw->nexthop));
        p.family = pw->af;
-       p.prefixlen = ((pw->af == AF_INET) ?
-                      IPV4_MAX_PREFIXLEN : IPV6_MAX_PREFIXLEN);
+       p.prefixlen =
+               ((pw->af == AF_INET) ? IPV4_MAX_PREFIXLEN : IPV6_MAX_BITLEN);
 
        afi = (pw->af == AF_INET) ? AFI_IP : AFI_IP6;
        table = zebra_vrf_table(afi, SAFI_UNICAST, pw->vrf_id);
index a4576b310e15c119f10fb4c8a0da257160f8a311..638a06d664c007573048e7ad9db04da18a7017fe 100644 (file)
@@ -662,7 +662,7 @@ static int nhlfe_nexthop_active_ipv6(zebra_nhlfe_t *nhlfe,
        /* Lookup nexthop in IPv6 routing table. */
        memset(&p, 0, sizeof(struct prefix_ipv6));
        p.family = AF_INET6;
-       p.prefixlen = IPV6_MAX_PREFIXLEN;
+       p.prefixlen = IPV6_MAX_BITLEN;
        p.prefix = nexthop->gate.ipv6;
 
        rn = route_node_match(table, (struct prefix *)&p);
index face0ef3bccfe7a79b4d24ca315251d3f854609c..d48dc68df6b3e0044febf4f5c43b0fc77301ba64 100644 (file)
@@ -2123,7 +2123,7 @@ static int nexthop_active(struct nexthop *nexthop, struct nhg_hash_entry *nhe,
                break;
        case AFI_IP6:
                p.family = AF_INET6;
-               p.prefixlen = IPV6_MAX_PREFIXLEN;
+               p.prefixlen = IPV6_MAX_BITLEN;
                p.u.prefix6 = nexthop->gate.ipv6;
                break;
        default:
index 73c2c3dda3a04bc3a5cbc05ba84f661699a6c78c..d26e15e27a7df0517890ad27b8d4edb92f60daef 100644 (file)
@@ -925,8 +925,8 @@ static const char *zebra_pbr_prefix2str(union prefixconstptr pu,
        const struct prefix *p = pu.p;
        char buf[PREFIX2STR_BUFFER];
 
-       if ((p->family == AF_INET && p->prefixlen == IPV4_MAX_PREFIXLEN) ||
-           (p->family == AF_INET6 && p->prefixlen == IPV6_MAX_PREFIXLEN)) {
+       if ((p->family == AF_INET && p->prefixlen == IPV4_MAX_PREFIXLEN)
+           || (p->family == AF_INET6 && p->prefixlen == IPV6_MAX_BITLEN)) {
                snprintf(str, size, "%s", inet_ntop(p->family, &p->u.prefix,
                                                    buf, PREFIX2STR_BUFFER));
                return str;
index d0acf779368a3055e35bafa78433296cb011ba6a..b25e31fd1fe2464d5652c38ba1cba4e087b92ac4 100644 (file)
@@ -352,7 +352,7 @@ struct route_entry *rib_match(afi_t afi, safi_t safi, vrf_id_t vrf_id,
                p.prefixlen = IPV4_MAX_PREFIXLEN;
        } else {
                p.u.prefix6 = addr->ipv6;
-               p.prefixlen = IPV6_MAX_PREFIXLEN;
+               p.prefixlen = IPV6_MAX_BITLEN;
        }
 
        rn = route_node_match(table, &p);