MPLS: Fix issue with recursive MPLS routes

Recursive MPLS routes used hostentry from the original route, which
triggered different table than MPLS table, and therefore were not
updated.

diff --git a/nest/mpls.c b/nest/mpls.c
index cd3a90d97c155748cac7af14e7185dc6e7f2c908..1992234ca9c23d2cded4b21e697003279a54c8ac 100644 (file)
--- a/nest/mpls.c
+++ b/nest/mpls.c
@@ -838,10 +838,13 @@ mpls_announce_fec(struct mpls_fec_map *m, struct mpls_fec *fec, const rta *src)
     const u32 *labels = &src->nh.label[src->nh.labels - src->nh.labels_orig];
     mpls_label_stack ms;
 
-    /* Apply the hostentry with the original labelstack */
+    /* Reconstruct the original labelstack */
     ms.len = src->nh.labels_orig;
     memcpy(ms.stack, labels, src->nh.labels_orig * sizeof(u32));
-    rta_apply_hostentry(a, src->hostentry, &ms);
+
+    /* The same hostentry, but different dependent table */
+    struct hostentry *s = src->hostentry;
+    rta_set_recursive_next_hop(m->channel->table, a, s->owner, s->addr, s->link, &ms);
   }
 
   net_addr_mpls n = NET_ADDR_MPLS(fec->label);

diff --git a/nest/route.h b/nest/route.h
index a9bcd7d3120a734f0a3b4dd2f74ebb0fa74c2295..0e502b99e746e0a4702ba8aad8f431decd7174c5 100644 (file)
--- a/nest/route.h
+++ b/nest/route.h
@@ -244,7 +244,8 @@ struct hostentry {
   ip_addr addr;                                /* IP address of host, part of key */
   ip_addr link;                                /* (link-local) IP address of host, used as gw
                                           if host is directly attached */
-  struct rtable *tab;                  /* Dependent table, part of key */
+  rtable *tab;                         /* Dependent table, part of key */
+  rtable *owner;                       /* Nexthop owner table */
   struct hostentry *next;              /* Next in hash chain */
   unsigned hash_key;                   /* Hash key */
   unsigned uc;                         /* Use count */

diff --git a/nest/rt-table.c b/nest/rt-table.c
index e1b2867f032ef7e18491e16818264c0f3edb2cd9..31c59ccb3a48ac3db89188101c3ea947d431dc65 100644 (file)
--- a/nest/rt-table.c
+++ b/nest/rt-table.c
@@ -3659,6 +3659,7 @@ rt_get_hostentry(rtable *tab, ip_addr a, ip_addr ll, rtable *dep)
       return he;
 
   he = hc_new_hostentry(hc, tab->rp, a, link, dep, k);
+  he->owner = tab;
   rt_update_hostentry(tab, he);
   return he;
 }