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

MFIB: coverity fixes

Browse Source 
Change-Id: I5c1bc498f7299e175070eb288e40f8d037c9be3a
Signed-off-by: Neale Ranns <nranns@cisco.com>
This commit is contained in:
Neale Ranns
2018-12-21 07:00:56 -08:00
committed by Florin Coras
parent d2bad81b0d
commit b7778b6310
2 changed files with 16 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/vnet/mfib/ip6_mfib.c
View File

@ -438,7 +438,7 @@ ip6_mfib_table_lookup (const ip6_mfib_t *mfib,
{
len = table->prefix_lengths_in_search_order[i];
ASSERT(len >= 0 && len <= 256);
ASSERT(len <= 256);
IP6_MFIB_MK_KEY(mfib, grp, src, len, key);
rv = clib_bihash_search_inline_2_40_8(&table->ip6_mhash, &key, &value);

21
src/vnet/mfib/mfib_entry.c
View File

@ -633,16 +633,25 @@ mfib_entry_stack (mfib_entry_t *mfib_entry,
{
/*
* for exclusive routes the source provided a replicate DPO
* we we stashed inthe special path list with one path
* which we stashed in the special path list with one path,
* so we can stack directly on that.
*/
ASSERT(1 == vec_len(ctx.next_hops));
dpo_stack(DPO_MFIB_ENTRY, dp,
&mfib_entry->mfe_rep,
&ctx.next_hops[0].path_dpo);
dpo_reset(&ctx.next_hops[0].path_dpo);
vec_free(ctx.next_hops);
if (NULL != ctx.next_hops)
{
dpo_stack(DPO_MFIB_ENTRY, dp,
&mfib_entry->mfe_rep,
&ctx.next_hops[0].path_dpo);
dpo_reset(&ctx.next_hops[0].path_dpo);
vec_free(ctx.next_hops);
}
else
{
dpo_stack(DPO_MFIB_ENTRY, dp,
&mfib_entry->mfe_rep,
drop_dpo_get(dp));
}
}
}
else