From e1ea6b20dfa61a628b3b5144b23cfdb06942a0be Mon Sep 17 00:00:00 2001 From: Igor Ryzhov Date: Tue, 25 May 2021 15:49:46 +0300 Subject: staticd: fix distance processing When the user adds the route + nexthop pair that already exists with a different distance, we should replace it instead of adding a new one. Likewise, when the user wants to delete the route + nexthop pair without explicitly entering the distance, we should delete the route. Fixes #8695. Signed-off-by: Igor Ryzhov --- staticd/static_vty.c | 43 +++++++++++++++++++++++++++++++++++++------ 1 file changed, 37 insertions(+), 6 deletions(-) (limited to 'staticd/static_vty.c') diff --git a/staticd/static_vty.c b/staticd/static_vty.c index 33a2728cd..3bdd4a673 100644 --- a/staticd/static_vty.c +++ b/staticd/static_vty.c @@ -153,6 +153,37 @@ static int static_route_leak(struct vty *vty, const char *svrf, static_get_nh_type(type, buf_nh_type, PREFIX_STRLEN); if (!negate) { + if (src_str) + snprintf(ab_xpath, sizeof(ab_xpath), + FRR_DEL_S_ROUTE_SRC_NH_KEY_NO_DISTANCE_XPATH, + "frr-staticd:staticd", "staticd", svrf, + buf_prefix, + yang_afi_safi_value2identity(afi, safi), + buf_src_prefix, table_id, buf_nh_type, nh_svrf, + buf_gate_str, ifname); + else + snprintf(ab_xpath, sizeof(ab_xpath), + FRR_DEL_S_ROUTE_NH_KEY_NO_DISTANCE_XPATH, + "frr-staticd:staticd", "staticd", svrf, + buf_prefix, + yang_afi_safi_value2identity(afi, safi), + table_id, buf_nh_type, nh_svrf, buf_gate_str, + ifname); + + /* + * If there's already the same nexthop but with a different + * distance, then remove it for the replacement. + */ + dnode = yang_dnode_get(vty->candidate_config->dnode, ab_xpath); + if (dnode) { + dnode = yang_get_subtree_with_no_sibling(dnode); + assert(dnode); + yang_dnode_get_path(dnode, ab_xpath, XPATH_MAXLEN); + + nb_cli_enqueue_change(vty, ab_xpath, NB_OP_DESTROY, + NULL); + } + /* route + path procesing */ if (src_str) snprintf(xpath_prefix, sizeof(xpath_prefix), @@ -277,20 +308,20 @@ static int static_route_leak(struct vty *vty, const char *svrf, } else { if (src_str) snprintf(ab_xpath, sizeof(ab_xpath), - FRR_DEL_S_ROUTE_SRC_NH_KEY_XPATH, + FRR_DEL_S_ROUTE_SRC_NH_KEY_NO_DISTANCE_XPATH, "frr-staticd:staticd", "staticd", svrf, buf_prefix, yang_afi_safi_value2identity(afi, safi), - buf_src_prefix, table_id, distance, - buf_nh_type, nh_svrf, buf_gate_str, ifname); + buf_src_prefix, table_id, buf_nh_type, nh_svrf, + buf_gate_str, ifname); else snprintf(ab_xpath, sizeof(ab_xpath), - FRR_DEL_S_ROUTE_NH_KEY_XPATH, + FRR_DEL_S_ROUTE_NH_KEY_NO_DISTANCE_XPATH, "frr-staticd:staticd", "staticd", svrf, buf_prefix, yang_afi_safi_value2identity(afi, safi), - table_id, distance, buf_nh_type, nh_svrf, - buf_gate_str, ifname); + table_id, buf_nh_type, nh_svrf, buf_gate_str, + ifname); dnode = yang_dnode_get(vty->candidate_config->dnode, ab_xpath); if (!dnode) { -- cgit v1.2.3