diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-03-31 22:38:29 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-03-31 22:38:29 +0200 |
| commit | 8cb129e9868eb5edb7bfe98917fad9931de6144b (patch) | |
| tree | f4758c2d2c43543949acb2e7dfbce885be704ef2 /pimd/pim_zebra.c | |
| parent | pimd: Cleanup unused variable write (diff) | |
| download | frr-8cb129e9868eb5edb7bfe98917fad9931de6144b.tar.xz frr-8cb129e9868eb5edb7bfe98917fad9931de6144b.zip | |
pimd: Cleanup api and stop crash
There is no need for a function that calls another function.
Additionally, nexthop_updates from zebra can be either
ZEBRA_NEXTHOP_UPDATE -or-
ZEBRA_IMPORT_CHECK_UPDATE
If we were to receive a IMPORT_CHECK_UPDATE the code
would cause a immediate crash. Fix this
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to '')
| -rw-r--r-- | pimd/pim_zebra.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/pimd/pim_zebra.c b/pimd/pim_zebra.c index cd7c3db59..4e18c478d 100644 --- a/pimd/pim_zebra.c +++ b/pimd/pim_zebra.c @@ -602,14 +602,6 @@ void sched_rpf_cache_refresh(void) 0, qpim_rpf_cache_refresh_delay_msec); } -static int -pim_zebra_nexthop_update (int command, struct zclient *zclient, - zebra_size_t length, vrf_id_t vrf_id) -{ - pim_parse_nexthop_update (zclient, command, vrf_id); - return 0; -} - static void pim_zebra_connected (struct zclient *zclient) { @@ -637,7 +629,7 @@ void pim_zebra_init(void) zclient->interface_down = pim_zebra_if_state_down; zclient->interface_address_add = pim_zebra_if_address_add; zclient->interface_address_delete = pim_zebra_if_address_del; - zclient->nexthop_update = pim_zebra_nexthop_update; + zclient->nexthop_update = pim_parse_nexthop_update; zclient_init(zclient, ZEBRA_ROUTE_PIM, 0); if (PIM_DEBUG_PIM_TRACE) { |