diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-04-01 15:41:32 -0400 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-04-02 08:20:36 -0400 |
| commit | ade155e146939906da26ce52f9edb1052fadd015 (patch) | |
| tree | 28a91b1de44e9f1597f16922ba040a2201e334ed /pimd/pim_cmd.c | |
| parent | c9cd7fbc3f4e0c7722897db71220ae997de1975f (diff) | |
pimd: pim_nexthop_lookup should return true/false
The current reverse logic led to this code construct
if (!pim_nexthop_lookup(...)) {
//Do something successfull
}
This is backwards and will cause logic errors when people
use this code. Fix to use true/false for success/failure.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_cmd.c')
| -rw-r--r-- | pimd/pim_cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pimd/pim_cmd.c b/pimd/pim_cmd.c index f9140802fb..3e61782f8d 100644 --- a/pimd/pim_cmd.c +++ b/pimd/pim_cmd.c @@ -5026,7 +5026,7 @@ DEFUN (show_ip_rib, return CMD_WARNING; } - if (pim_nexthop_lookup(vrf->info, &nexthop, addr, 0)) { + if (!pim_nexthop_lookup(vrf->info, &nexthop, addr, 0)) { vty_out(vty, "Failure querying RIB nexthop for unicast address %s\n", addr_str); |