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
@@ -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;
