diff options
| author | paco <paco@voltanet.io> | 2018-06-26 21:19:11 +0200 |
|---|---|---|
| committer | paco <paco@voltanet.io> | 2018-06-28 11:50:49 +0200 |
| commit | a35fb78f1cd4b32c433a9ba7258493328ded97a8 (patch) | |
| tree | dbce303af5a7317bd83aec2ff24b417f4dc857cd /ospf6d | |
| parent | Merge pull request #2549 from pacovn/Coverity_1452317_Explicit_null_dereferenced (diff) | |
| download | frr-a35fb78f1cd4b32c433a9ba7258493328ded97a8.tar.xz frr-a35fb78f1cd4b32c433a9ba7258493328ded97a8.zip | |
ospf6d: dead code (Coverity 1399231)
It should have the same behavior when debug not enabled, and slightly
different behavior when debug enabled (previously dead code now should
show debug messages in debug mode)
Signed-off-by: F. Aragon <paco@voltanet.io>
Diffstat (limited to 'ospf6d')
| -rw-r--r-- | ospf6d/ospf6_abr.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/ospf6d/ospf6_abr.c b/ospf6d/ospf6_abr.c index b3aa3b21d..3ad321dc6 100644 --- a/ospf6d/ospf6_abr.c +++ b/ospf6d/ospf6_abr.c @@ -161,9 +161,10 @@ int ospf6_abr_originate_summary_to_area(struct ospf6_route *route, && route->type != OSPF6_DEST_TYPE_RANGE && ((route->type != OSPF6_DEST_TYPE_ROUTER) || !CHECK_FLAG(route->path.router_bits, OSPF6_ROUTER_BIT_E))) { - if (is_debug) - zlog_debug( - "Route type is none of network, range nor ASBR, ignore"); +#if 0 + zlog_debug( + "Route type is none of network, range nor ASBR, ignore"); +#endif return 0; } @@ -177,16 +178,17 @@ int ospf6_abr_originate_summary_to_area(struct ospf6_route *route, /* do not generate if the path's area is the same as target area */ if (route->path.area_id == area->area_id) { - if (is_debug) - zlog_debug("The route is in the area itself, ignore"); +#if 0 + zlog_debug("The route is in the area itself, ignore"); +#endif return 0; } /* do not generate if the nexthops belongs to the target area */ if (ospf6_abr_nexthops_belong_to_area(route, area)) { - if (is_debug) - zlog_debug( - "The route's nexthop is in the same area, ignore"); +#if 0 + zlog_debug("The route's nexthop is in the same area, ignore"); +#endif return 0; } |