summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEyal Bari <ebari@cisco.com>2017-03-02 10:43:19 +0200
committerJohn Lo <loj@cisco.com>2017-03-02 19:43:18 +0000
commit25b36674f7e9072084f8f149067450f5eb6a5841 (patch)
tree9221d46f59f60820cd2428404f4e20bf3481864b
parent23caa885afad3501ea95b52cdbc64c0d48072a83 (diff)
bridge domain: fix members reordered when removing
since adding support for multicast vxlan flooding (flood class tunnel master) correct flood functionality depends on the order of the memebers vector solved by using vec_delete instead of vec_del1 which swaps members before deleting the last element Change-Id: I234f218d49172b4142c567db9699a5cb274e4a66 Signed-off-by: Eyal Bari <ebari@cisco.com>
-rw-r--r--src/vnet/l2/l2_bd.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/vnet/l2/l2_bd.c b/src/vnet/l2/l2_bd.c
index 22f83d0b3e4..f741b64371f 100644
--- a/src/vnet/l2/l2_bd.c
+++ b/src/vnet/l2/l2_bd.c
@@ -185,7 +185,7 @@ bd_remove_member (l2_bridge_domain_t * bd_config, u32 sw_if_index)
else if (sw_if->flood_class == VNET_FLOOD_CLASS_TUNNEL_NORMAL)
bd_config->tun_normal_count--;
}
- vec_del1 (bd_config->members, ix);
+ vec_delete (bd_config->members, 1, ix);
update_flood_count (bd_config);
return BD_REMOVE_ERROR_OK;