diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-05-18 22:53:50 -0400 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-07-24 13:51:35 -0400 |
| commit | cb9c7c50d3928c8857b761ce1579b194736ece7a (patch) | |
| tree | 9a2f5b41b9d5ee2aa65f2d338e7e7c3ec0e4f2a6 /pimd/pim_cmd.c | |
| parent | da82728dbf709881ad21464ee239ba5e32d196dc (diff) | |
pimd: Refactor some functions
A bunch of functions had return values that were never
checked for ( and not needed ) and opposite return values
for proper calling function boolean logic.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_cmd.c')
| -rw-r--r-- | pimd/pim_cmd.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/pimd/pim_cmd.c b/pimd/pim_cmd.c index 96cbcd1844..ce51edf0bd 100644 --- a/pimd/pim_cmd.c +++ b/pimd/pim_cmd.c @@ -3599,10 +3599,9 @@ DEFUN (show_ip_pim_nexthop_lookup, grp.u.prefix4 = grp_addr; memset(&nexthop, 0, sizeof(nexthop)); - if ((pim_find_or_track_nexthop(&nht_p, NULL, NULL, &pnc)) == 1) { - // Compute PIM RPF using Cached nexthop + if (pim_find_or_track_nexthop(&nht_p, NULL, NULL, &pnc)) pim_ecmp_nexthop_search(&pnc, &nexthop, &nht_p, &grp, 0); - } else + else pim_ecmp_nexthop_lookup(&nexthop, vif_source, &nht_p, &grp, 0); pim_addr_dump("<grp?>", &grp, grp_str, sizeof(grp_str)); |