diff options
author | Alex Wang <alexw@nicira.com> | 2015-06-08 22:57:09 -0700 |
---|---|---|
committer | Alex Wang <alexw@nicira.com> | 2015-06-11 16:27:32 -0700 |
commit | e815de429c66a6c73d0605e36bf9be4eeec785ce (patch) | |
tree | bdad065bd8124a9ddba524d1200af26f383f7c40 /vtep | |
parent | 6787a49f69168e78f5d8af8aa8be128385650a50 (diff) | |
download | openvswitch-e815de429c66a6c73d0605e36bf9be4eeec785ce.tar.gz |
vtep-ctl: Fix a bug.
add_port_to_cache() uses 'cache_name' as the shash node name for
shash_add(). So, the del_cached_port() must also pass 'cache_name'
as argument for shash_find_and_delete().
This bug does not cause any issue currently but should be fixed.
Signed-off-by: Alex Wang <alexw@nicira.com>
Acked-by: Justin Pettit <jpettit@nicira.com>
Diffstat (limited to 'vtep')
-rw-r--r-- | vtep/vtep-ctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vtep/vtep-ctl.c b/vtep/vtep-ctl.c index ead22ea39..a501a9825 100644 --- a/vtep/vtep-ctl.c +++ b/vtep/vtep-ctl.c @@ -798,7 +798,7 @@ del_cached_port(struct vtep_ctl_context *ctx, struct vtep_ctl_port *port) char *cache_name = xasprintf("%s+%s", port->ps->name, port->port_cfg->name); list_remove(&port->ports_node); - shash_find_and_delete(&ctx->ports, port->port_cfg->name); + shash_find_and_delete(&ctx->ports, cache_name); vteprec_physical_port_delete(port->port_cfg); free(cache_name); free(port); |