Browse Source

bgpd: IPv6 Multipath broken with nexthop global address

IPv6 multipath is broken in BGP if nexthop contains only global address.
IPv6 always uses both nextop IPv6 address and ifIndex in sending routes down to
zebra. In cases where only the global IPv6 address is present in the nexthop
information, the existing code doesn't set the ifIndex. An example of such a
case is when a route-map isused with "set ipv6 next-hop" and only global
address is specified. This code causes the ifIndex to be determined and
set thereby fixing the multipath programming.

Signed-off-by: Dinesh G Dutt <ddutt@cumulusnetworks.com>
Reviewed-by: Shrijeet Mukherjee <shm@cumulusnetworks.com>
Dinesh Dutt 4 years ago
parent
commit
4feb0d02c0
1 changed files with 7 additions and 4 deletions
  1. 7 4
      bgpd/bgp_zebra.c

+ 7 - 4
bgpd/bgp_zebra.c

@@ -816,7 +816,7 @@ bgp_zebra_announce (struct prefix *p, struct bgp_info *info, struct bgp *bgp, sa
       if (nexthop == NULL)
 	return;
 
-      if (IN6_IS_ADDR_LINKLOCAL (nexthop) && ! ifindex)
+      if (!ifindex)
 	{
 	  if (info->peer->ifname)
 	    ifindex = ifname2ifindex (info->peer->ifname);
@@ -830,11 +830,12 @@ bgp_zebra_announce (struct prefix *p, struct bgp_info *info, struct bgp *bgp, sa
       for (mpinfo = bgp_info_mpath_first (info); mpinfo;
            mpinfo = bgp_info_mpath_next (mpinfo))
 	{
+	  ifindex = 0;
+
           /* Only global address nexthop exists. */
           if (mpinfo->attr->extra->mp_nexthop_len == 16)
-            {
               nexthop = &mpinfo->attr->extra->mp_nexthop_global;
-            }
+
           /* If both global and link-local address present. */
 	  if (mpinfo->attr->extra->mp_nexthop_len == 32)
             {
@@ -854,12 +855,13 @@ bgp_zebra_announce (struct prefix *p, struct bgp_info *info, struct bgp *bgp, sa
                   ifindex = mpinfo->peer->nexthop.ifp->ifindex;
                 }
             }
+
 	  if (nexthop == NULL)
 	    {
 	      continue;
 	    }
 
-          if (IN6_IS_ADDR_LINKLOCAL (nexthop) && ! ifindex)
+          if (!ifindex)
 	    {
 	      if (mpinfo->peer->ifname)
                 {
@@ -870,6 +872,7 @@ bgp_zebra_announce (struct prefix *p, struct bgp_info *info, struct bgp *bgp, sa
 		  ifindex = mpinfo->peer->nexthop.ifp->ifindex;
 		}
 	    }
+
 	  if (ifindex == 0)
 	    {
 	      continue;