diff options
| author | Igor Ryzhov <iryzhov@nfware.com> | 2024-02-25 21:50:41 +0200 | 
|---|---|---|
| committer | Mergify <37929162+mergify[bot]@users.noreply.github.com> | 2024-02-26 07:44:44 +0000 | 
| commit | 7dd059f6f3de5c358667a028241a9c71f9358c81 (patch) | |
| tree | ade3acd7a4560e011025cc454e415c3bee24d48c /bgpd/bgp_routemap_nb.c | |
| parent | 3496e9b20538a76584f8a453e764b3daaea44bf1 (diff) | |
bgpd, yang: fix missing mandatory/default statements on some leafs
The code expects these leafs to always exist. If they are not set, the
daemon would crash. CLI always sets them, but we should properly mark
them as mandatory/default to prevent them from being missed when using
the API.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
(cherry picked from commit 83a3b0f10f994ee5adbbef0f6bd970b69eab4486)
Diffstat (limited to 'bgpd/bgp_routemap_nb.c')
| -rw-r--r-- | bgpd/bgp_routemap_nb.c | 5 | 
1 files changed, 0 insertions, 5 deletions
diff --git a/bgpd/bgp_routemap_nb.c b/bgpd/bgp_routemap_nb.c index 2320d2f908..096502aaa9 100644 --- a/bgpd/bgp_routemap_nb.c +++ b/bgpd/bgp_routemap_nb.c @@ -156,7 +156,6 @@ const struct frr_yang_module_info frr_bgp_route_map_info = {  			.xpath = "/frr-route-map:lib/route-map/entry/match-condition/rmap-match-condition/frr-bgp-route-map:comm-list/comm-list-name",  			.cbs = {  				.modify = lib_route_map_entry_match_condition_rmap_match_condition_comm_list_comm_list_name_modify, -				.destroy = lib_route_map_entry_match_condition_rmap_match_condition_comm_list_comm_list_name_destroy,  			}  		},  		{ @@ -367,14 +366,12 @@ const struct frr_yang_module_info frr_bgp_route_map_info = {  			.xpath = "/frr-route-map:lib/route-map/entry/set-action/rmap-set-action/frr-bgp-route-map:aggregator/aggregator-asn",  			.cbs = {  				.modify = lib_route_map_entry_set_action_rmap_set_action_aggregator_aggregator_asn_modify, -				.destroy = lib_route_map_entry_set_action_rmap_set_action_aggregator_aggregator_asn_destroy,  			}  		},  		{  			.xpath = "/frr-route-map:lib/route-map/entry/set-action/rmap-set-action/frr-bgp-route-map:aggregator/aggregator-address",  			.cbs = {  				.modify = lib_route_map_entry_set_action_rmap_set_action_aggregator_aggregator_address_modify, -				.destroy = lib_route_map_entry_set_action_rmap_set_action_aggregator_aggregator_address_destroy,  			}  		},  		{ @@ -403,7 +400,6 @@ const struct frr_yang_module_info frr_bgp_route_map_info = {  			.xpath = "/frr-route-map:lib/route-map/entry/set-action/rmap-set-action/frr-bgp-route-map:extcommunity-lb/lb-type",  			.cbs = {  				.modify = lib_route_map_entry_set_action_rmap_set_action_extcommunity_lb_lb_type_modify, -				.destroy = lib_route_map_entry_set_action_rmap_set_action_extcommunity_lb_lb_type_destroy,  			}  		},  		{ @@ -424,7 +420,6 @@ const struct frr_yang_module_info frr_bgp_route_map_info = {  			.xpath = "/frr-route-map:lib/route-map/entry/set-action/rmap-set-action/frr-bgp-route-map:extcommunity-lb/two-octet-as-specific",  			.cbs = {  				.modify = lib_route_map_entry_set_action_rmap_set_action_extcommunity_lb_two_octet_as_specific_modify, -				.destroy = lib_route_map_entry_set_action_rmap_set_action_extcommunity_lb_two_octet_as_specific_destroy,  			}  		},  		{  | 
