Mirror/frr
0
0
Fork 1
mirror of https://github.com/FRRouting/frr.git synced 2025-05-01 05:57:15 +02:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

ripngd: use correct autocompletion for distribute-list commands

Browse source 
Currently, we always use access-list autocompletion, even if configuring
prefix-lists. We should differentiate. Also, fix missing "IPv6"
docstring and use correct address family.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
This commit is contained in:
Igor Ryzhov 2024-01-22 23:44:17 +02:00
parent 45b868a2af
commit 83a698d891
1 changed files with 69 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

79
ripngd/ripng_cli.c
View file

@ -518,10 +518,10 @@ DEFPY_YANG (clear_ipv6_rip,
DEFPY_YANG( DEFPY_YANG(
ripng_ipv6_distribute_list, ripng_ipv6_distribute_list_cmd, ripng_ipv6_distribute_list, ripng_ipv6_distribute_list_cmd,
"ipv6 distribute-list [prefix]$prefix ACCESSLIST4_NAME$name <in|out>$dir [WORD$ifname]", "ipv6 distribute-list ACCESSLIST6_NAME$name <in|out>$dir [WORD$ifname]",
"IPv6\n"
"Filter networks in routing updates\n" "Filter networks in routing updates\n"
"Specify a prefix list\n" "Access-list name\n"
"access-list or prefix-list name\n"
"Filter incoming routing updates\n" "Filter incoming routing updates\n"
"Filter outgoing routing updates\n" "Filter outgoing routing updates\n"
"Interface name\n") "Interface name\n")
@ -529,8 +529,29 @@ DEFPY_YANG(
char xpath[XPATH_MAXLEN]; char xpath[XPATH_MAXLEN];
snprintf(xpath, sizeof(xpath), snprintf(xpath, sizeof(xpath),
"./distribute-list[interface='%s']/%s/%s-list", "./distribute-list[interface='%s']/%s/access-list",
ifname ? ifname : "", dir, prefix ? "prefix" : "access"); ifname ? ifname : "", dir);
/* nb_cli_enqueue_change(vty, ".", NB_OP_CREATE, NULL); */
nb_cli_enqueue_change(vty, xpath, NB_OP_MODIFY, name);
return nb_cli_apply_changes(vty, NULL);
}
DEFPY_YANG(
ripng_ipv6_distribute_list_prefix, ripng_ipv6_distribute_list_prefix_cmd,
"ipv6 distribute-list prefix PREFIXLIST6_NAME$name <in|out>$dir [WORD$ifname]",
"IPv6\n"
"Filter networks in routing updates\n"
"Specify a prefix list\n"
"Prefix-list name\n"
"Filter incoming routing updates\n"
"Filter outgoing routing updates\n"
"Interface name\n")
{
char xpath[XPATH_MAXLEN];
snprintf(xpath, sizeof(xpath),
"./distribute-list[interface='%s']/%s/prefix-list",
ifname ? ifname : "", dir);
/* nb_cli_enqueue_change(vty, ".", NB_OP_CREATE, NULL); */ /* nb_cli_enqueue_change(vty, ".", NB_OP_CREATE, NULL); */
nb_cli_enqueue_change(vty, xpath, NB_OP_MODIFY, name); nb_cli_enqueue_change(vty, xpath, NB_OP_MODIFY, name);
return nb_cli_apply_changes(vty, NULL); return nb_cli_apply_changes(vty, NULL);
@ -538,11 +559,11 @@ DEFPY_YANG(
DEFPY_YANG(no_ripng_ipv6_distribute_list, DEFPY_YANG(no_ripng_ipv6_distribute_list,
no_ripng_ipv6_distribute_list_cmd, no_ripng_ipv6_distribute_list_cmd,
"no ipv6 distribute-list [prefix]$prefix [ACCESSLIST4_NAME$name] <in|out>$dir [WORD$ifname]", "no ipv6 distribute-list [ACCESSLIST6_NAME$name] <in|out>$dir [WORD$ifname]",
NO_STR NO_STR
"IPv6\n"
"Filter networks in routing updates\n" "Filter networks in routing updates\n"
"Specify a prefix list\n" "Access-list name\n"
"access-list or prefix-list name\n"
"Filter incoming routing updates\n" "Filter incoming routing updates\n"
"Filter outgoing routing updates\n" "Filter outgoing routing updates\n"
"Interface name\n") "Interface name\n")
@ -551,8 +572,44 @@ DEFPY_YANG(no_ripng_ipv6_distribute_list,
char xpath[XPATH_MAXLEN]; char xpath[XPATH_MAXLEN];
snprintf(xpath, sizeof(xpath), snprintf(xpath, sizeof(xpath),
"./distribute-list[interface='%s']/%s/%s-list", "./distribute-list[interface='%s']/%s/access-list",
ifname ? ifname : "", dir, prefix ? "prefix" : "access"); ifname ? ifname : "", dir);
/*
* See if the user has specified specific list so check it exists.
*
* NOTE: Other FRR CLI commands do not do this sort of verification and
* there may be an official decision not to.
*/
if (name) {
value_node = yang_dnode_getf(vty->candidate_config->dnode, "%s/%s",
VTY_CURR_XPATH, xpath);
if (!value_node || strcmp(name, lyd_get_value(value_node))) {
vty_out(vty, "distribute list doesn't exist\n");
return CMD_WARNING_CONFIG_FAILED;
}
}
nb_cli_enqueue_change(vty, xpath, NB_OP_DESTROY, NULL);
return nb_cli_apply_changes(vty, NULL);
}
DEFPY_YANG(no_ripng_ipv6_distribute_list_prefix,
no_ripng_ipv6_distribute_list_prefix_cmd,
"no ipv6 distribute-list prefix [PREFIXLIST6_NAME$name] <in|out>$dir [WORD$ifname]",
NO_STR
"IPv6\n"
"Filter networks in routing updates\n"
"Specify a prefix list\n"
"Prefix-list name\n"
"Filter incoming routing updates\n"
"Filter outgoing routing updates\n"
"Interface name\n")
{
const struct lyd_node *value_node;
char xpath[XPATH_MAXLEN];
snprintf(xpath, sizeof(xpath),
"./distribute-list[interface='%s']/%s/prefix-list",
ifname ? ifname : "", dir);
/* /*
* See if the user has specified specific list so check it exists. * See if the user has specified specific list so check it exists.
* *
@ -577,7 +634,9 @@ void ripng_cli_init(void)
install_element(CONFIG_NODE, &no_router_ripng_cmd); install_element(CONFIG_NODE, &no_router_ripng_cmd);
install_element(RIPNG_NODE, &ripng_ipv6_distribute_list_cmd); install_element(RIPNG_NODE, &ripng_ipv6_distribute_list_cmd);
install_element(RIPNG_NODE, &ripng_ipv6_distribute_list_prefix_cmd);
install_element(RIPNG_NODE, &no_ripng_ipv6_distribute_list_cmd); install_element(RIPNG_NODE, &no_ripng_ipv6_distribute_list_cmd);
install_element(RIPNG_NODE, &no_ripng_ipv6_distribute_list_prefix_cmd);
install_element(RIPNG_NODE, &ripng_allow_ecmp_cmd); install_element(RIPNG_NODE, &ripng_allow_ecmp_cmd);
install_element(RIPNG_NODE, &no_ripng_allow_ecmp_cmd); install_element(RIPNG_NODE, &no_ripng_allow_ecmp_cmd);