diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-02-18 23:23:50 +0100 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-02-23 13:08:36 +0100 |
| commit | 783fc3cd45eb2160304b749535c61f8830d9e1ba (patch) | |
| tree | d22a5687df4012f894dd742643e738240ab96f54 /zebra/zebra_ns.c | |
| parent | *: Make assignment from RB_ROOT in while loop work better (diff) | |
| download | frr-783fc3cd45eb2160304b749535c61f8830d9e1ba.tar.xz frr-783fc3cd45eb2160304b749535c61f8830d9e1ba.zip | |
zebra: Fix warning found in CI system
The Clang SA system found a new issue:
Dead store: Dead assignment.
This fixes that issue
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'zebra/zebra_ns.c')
| -rw-r--r-- | zebra/zebra_ns.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/zebra/zebra_ns.c b/zebra/zebra_ns.c index e8bdadee5..c7561f426 100644 --- a/zebra/zebra_ns.c +++ b/zebra/zebra_ns.c @@ -124,7 +124,7 @@ struct route_table *zebra_ns_get_table(struct zebra_ns *zns, return znst->table; } -static struct zebra_ns_table *zebra_ns_free_table(struct zebra_ns_table *znst) +static void zebra_ns_free_table(struct zebra_ns_table *znst) { void *table_info; rib_close_table(znst->table); @@ -133,7 +133,6 @@ static struct zebra_ns_table *zebra_ns_free_table(struct zebra_ns_table *znst) route_table_finish(znst->table); XFREE(MTYPE_RIB_TABLE_INFO, table_info); XFREE(MTYPE_ZEBRA_NS, znst); - return NULL; } int zebra_ns_disable(ns_id_t ns_id, void **info) @@ -145,7 +144,7 @@ int zebra_ns_disable(ns_id_t ns_id, void **info) znst = RB_ROOT(zebra_ns_table_head, &zns->ns_tables); RB_REMOVE(zebra_ns_table_head, &zns->ns_tables, znst); - znst = zebra_ns_free_table(znst); + zebra_ns_free_table(znst); } route_table_finish(zns->if_table); zebra_vxlan_ns_disable(zns); |