Check change in unnumbered setting before updating IP enabled state

[vpp.git] / src / vnet / interface_cli.c

diff --git a/src/vnet/interface_cli.c b/src/vnet/interface_cli.c
index ec8530d..c156655 100644 (file)
--- a/src/vnet/interface_cli.c
+++ b/src/vnet/interface_cli.c
@@ -844,6 +844,7 @@ set_unnumbered (vlib_main_t * vm,
   vnet_sw_interface_t *si;
   int is_set = 0;
   int is_del = 0;
+  u32 was_unnum;
 
   if (unformat (input, "%U use %U",
                unformat_vnet_sw_interface, vnm, &unnumbered_sw_if_index,
@@ -858,12 +859,13 @@ set_unnumbered (vlib_main_t * vm,
                              format_unformat_error, input);
 
   si = vnet_get_sw_interface (vnm, unnumbered_sw_if_index);
+  was_unnum = (si->flags & VNET_SW_INTERFACE_FLAG_UNNUMBERED);
+
   if (is_del)
     {
       si->flags &= ~(VNET_SW_INTERFACE_FLAG_UNNUMBERED);
       si->unnumbered_sw_if_index = (u32) ~ 0;
-      ip4_sw_interface_enable_disable (unnumbered_sw_if_index, 0);
-      ip6_sw_interface_enable_disable (unnumbered_sw_if_index, 0);
+
       ip4_main.lookup_main.if_address_pool_index_by_sw_if_index
        [unnumbered_sw_if_index] = ~0;
       ip6_main.lookup_main.if_address_pool_index_by_sw_if_index
@@ -873,8 +875,7 @@ set_unnumbered (vlib_main_t * vm,
     {
       si->flags |= VNET_SW_INTERFACE_FLAG_UNNUMBERED;
       si->unnumbered_sw_if_index = inherit_from_sw_if_index;
-      ip4_sw_interface_enable_disable (unnumbered_sw_if_index, 1);
-      ip6_sw_interface_enable_disable (unnumbered_sw_if_index, 1);
+
       ip4_main.lookup_main.if_address_pool_index_by_sw_if_index
        [unnumbered_sw_if_index] =
        ip4_main.lookup_main.if_address_pool_index_by_sw_if_index
@@ -884,6 +885,16 @@ set_unnumbered (vlib_main_t * vm,
        ip6_main.lookup_main.if_address_pool_index_by_sw_if_index
        [inherit_from_sw_if_index];
     }
+  else
+    {
+      return (0);
+    }
+
+  if (was_unnum != (si->flags & VNET_SW_INTERFACE_FLAG_UNNUMBERED))
+    {
+      ip4_sw_interface_enable_disable (unnumbered_sw_if_index, !is_del);
+      ip6_sw_interface_enable_disable (unnumbered_sw_if_index, !is_del);
+    }
 
   return 0;
 }