diff options
| author |   Neale Ranns <neale.ranns@cisco.com> | 2018-03-19 02:32:57 -0700 |
|---|---|---|
| committer |   Chris Luke <chris_luke@comcast.com> | 2018-03-19 17:10:41 +0000 |
| commit | 19bd1902cb8cc074470f1e178d1df0601f75f58b (patch) | |
| tree | 17598f518775aed95225545fae360ab432175d6f /src/vnet/mpls | |
| parent | 9a1f49ef30c6c07ce1254fa57a3d024f9b05f523 (diff) | |
Coverity found bugs in recent MPLS changes
Change-Id: I590945fdc1af53208c990a52bbecdc992fd27532
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
Diffstat (limited to 'src/vnet/mpls')
| -rw-r--r-- | src/vnet/mpls/mpls_tunnel.c | 21 |
1 files changed, 6 insertions, 15 deletions
diff --git a/src/vnet/mpls/mpls_tunnel.c b/src/vnet/mpls/mpls_tunnel.c index c2067d81b00..7ca2c12cbcf 100644 --- a/src/vnet/mpls/mpls_tunnel.c +++ b/src/vnet/mpls/mpls_tunnel.c @@ -129,23 +129,14 @@ mpls_tunnel_collect_forwarding (fib_node_index_t pl_index, */ path_ext->fpe_mpls_flags |= FIB_PATH_EXT_MPLS_FLAG_NO_IP_TTL_DECR; - if (NULL != path_ext) - { - /* - * found a matching extension. stack it to obtain the forwarding - * info for this path. - */ - ctx->next_hops = fib_path_ext_stack(path_ext, - ctx->fct, - ctx->fct, - ctx->next_hops); - } - else - ASSERT(0); /* - * else - * There should be a path-extenios associated with each path + * found a matching extension. stack it to obtain the forwarding + * info for this path. */ + ctx->next_hops = fib_path_ext_stack(path_ext, + ctx->fct, + ctx->fct, + ctx->next_hops); return (FIB_PATH_LIST_WALK_CONTINUE); } |