commit 324ed1f87aedf30a45aea951edb2e717c5f9b7fb
author Avneesh Sachdev <avneesh@opensourcerouting.org> Tue Nov 13 22:49:01 2012 +0000
committer David Lamparter <equinox@opensourcerouting.org> Fri Nov 30 21:41:17 2012 +0100
tree 3e23528caf97c699bc2403a7e87443005f87f446
parent b9c24cd23091e817942d4eaab95712e84dbec8aa

zebra: include intf when sending IPv4 nexthop to FPM

* zebra/zebra_fpm_netlink.c

    Change the zebra FPM code to include an interface index when
    encoding a nexthop even if the protocol only provided a gateway
    address (e.g, NEXTHOP_TYPE_IPV4).

Signed-off-by: Avneesh Sachdev <avneesh@opensourcerouting.org>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

zebra/zebra_fpm_netlink.c

diff --git a/zebra/zebra_fpm_netlink.c b/zebra/zebra_fpm_netlink.c
index 90d3afb..67bcf0a 100644
--- a/zebra/zebra_fpm_netlink.c
+++ b/zebra/zebra_fpm_netlink.c
@@ -167,6 +167,7 @@
     {
       nhi.recursive = 1;
       nhi.type = nexthop->rtype;
+      nhi.if_index = nexthop->rifindex;
 
       if (nexthop->rtype == NEXTHOP_TYPE_IPV4
 	  || nexthop->rtype == NEXTHOP_TYPE_IPV4_IFINDEX)
@@ -186,15 +187,9 @@
 #endif /* HAVE_IPV6 */
 
       if (nexthop->rtype == NEXTHOP_TYPE_IFINDEX
-	  || nexthop->rtype == NEXTHOP_TYPE_IFNAME
-	  || nexthop->rtype == NEXTHOP_TYPE_IPV4_IFINDEX
-	  || nexthop->rtype == NEXTHOP_TYPE_IPV6_IFINDEX
-	  || nexthop->rtype == NEXTHOP_TYPE_IPV6_IFNAME)
+	  || nexthop->rtype == NEXTHOP_TYPE_IFNAME)
 	{
-	  nhi.if_index = nexthop->rifindex;
-	  if ((nexthop->rtype == NEXTHOP_TYPE_IPV4_IFINDEX
-	       || nexthop->rtype == NEXTHOP_TYPE_IFINDEX)
-	      && nexthop->src.ipv4.s_addr)
+	  if (nexthop->src.ipv4.s_addr)
 	    src = &nexthop->src;
 	}
 
@@ -203,6 +198,7 @@
 
   nhi.recursive = 0;
   nhi.type = nexthop->type;
+  nhi.if_index = nexthop->ifindex;
 
   if (nexthop->type == NEXTHOP_TYPE_IPV4
       || nexthop->type == NEXTHOP_TYPE_IPV4_IFINDEX)
@@ -220,20 +216,13 @@
       nhi.gateway = &nexthop->gate;
     }
 #endif /* HAVE_IPV6 */
-  if (nexthop->type == NEXTHOP_TYPE_IFINDEX
-      || nexthop->type == NEXTHOP_TYPE_IFNAME
-      || nexthop->type == NEXTHOP_TYPE_IPV4_IFINDEX)
-    {
-      nhi.if_index = nexthop->ifindex;
 
+  if (nexthop->type == NEXTHOP_TYPE_IFINDEX
+      || nexthop->type == NEXTHOP_TYPE_IFNAME)
+    {
       if (nexthop->src.ipv4.s_addr)
 	src = &nexthop->src;
     }
-  else if (nexthop->type == NEXTHOP_TYPE_IPV6_IFINDEX
-	   || nexthop->type == NEXTHOP_TYPE_IPV6_IFNAME)
-    {
-      nhi.if_index = nexthop->ifindex;
-    }
 
   /*
    * Fall through...