ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ioam: do not reuse existing vnet symbol

Browse Source 
vxlan_gpe_init() is already defined in libvnet. When loading ioam plugin
we end up having 2 different objects using the same symbol.
ASan in GCC-10 started to enforce the One-Definition-Rule and it seems
like good hygiene anyway.

Type: fix

Change-Id: I2ea9af1821bca6482a290742e9a109fc25692f37
Signed-off-by: Benoît Ganne <bganne@cisco.com>
This commit is contained in:
Benoît Ganne
2020-04-24 09:20:13 +02:00
committed by Dave Barach
parent a66971f980
commit 83ceffcd98
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/plugins/ioam/lib-vxlan-gpe/vxlan_gpe_api.c
View File

@ -204,7 +204,7 @@ static void vl_api_vxlan_gpe_ioam_transit_disable_t_handler
#include <ioam/lib-vxlan-gpe/ioam_vxlan_gpe.api.c> #include <ioam/lib-vxlan-gpe/ioam_vxlan_gpe.api.c>
static clib_error_t * static clib_error_t *
vxlan_gpe_init (vlib_main_t * vm) ioam_vxlan_gpe_init (vlib_main_t * vm)
{ {
vxlan_gpe_ioam_main_t *sm = &vxlan_gpe_ioam_main; vxlan_gpe_ioam_main_t *sm = &vxlan_gpe_ioam_main;
u32 encap_node_index = vxlan_gpe_encap_ioam_v4_node.index; u32 encap_node_index = vxlan_gpe_encap_ioam_v4_node.index;
@ -242,7 +242,7 @@ vxlan_gpe_init (vlib_main_t * vm)
return 0; return 0;
} }
VLIB_INIT_FUNCTION (vxlan_gpe_init); VLIB_INIT_FUNCTION (ioam_vxlan_gpe_init);
/* /*
* fd.io coding-style-patch-verification: ON * fd.io coding-style-patch-verification: ON