matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

isisd: fix change flex-algorithm number from uint32 to uint8

Browse source 
The algorithm number is encoded on 8 bits and does not require
an unsigned 32 bit value to store the value.

Fixes: cc4926c128 ("isisd,yang: add algorithm-prefix-sid configuration tree")

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
(cherry picked from commit 8e861a75e8)
This commit is contained in:
Philippe Guibert 2024-10-25 10:23:54 +02:00 committed by Mergify
parent ecaada19cf
commit e61379f38c
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
isisd/isis_cli.c
View file

@ -2015,12 +2015,12 @@ void cli_show_isis_prefix_sid_algorithm(struct vty *vty,
const char *sid_value_type; const char *sid_value_type;
const char *sid_value; const char *sid_value;
bool n_flag_clear; bool n_flag_clear;
uint32_t algorithm; uint8_t algorithm;
prefix = yang_dnode_get_string(dnode, "prefix"); prefix = yang_dnode_get_string(dnode, "prefix");
sid_value_type = yang_dnode_get_string(dnode, "sid-value-type"); sid_value_type = yang_dnode_get_string(dnode, "sid-value-type");
sid_value = yang_dnode_get_string(dnode, "sid-value"); sid_value = yang_dnode_get_string(dnode, "sid-value");
algorithm = yang_dnode_get_uint32(dnode, "algo"); algorithm = yang_dnode_get_uint8(dnode, "algo");
lh_behavior = yang_dnode_get_string(dnode, "last-hop-behavior"); lh_behavior = yang_dnode_get_string(dnode, "last-hop-behavior");
n_flag_clear = yang_dnode_get_bool(dnode, "n-flag-clear"); n_flag_clear = yang_dnode_get_bool(dnode, "n-flag-clear");

4
isisd/isis_nb_config.c
View file

@ -2630,14 +2630,14 @@ int isis_instance_segment_routing_algorithm_prefix_sid_create(
struct isis_area *area; struct isis_area *area;
struct prefix prefix; struct prefix prefix;
struct sr_prefix_cfg *pcfg; struct sr_prefix_cfg *pcfg;
uint32_t algorithm; uint8_t algorithm;
if (args->event != NB_EV_APPLY) if (args->event != NB_EV_APPLY)
return NB_OK; return NB_OK;
area = nb_running_get_entry(args->dnode, NULL, true); area = nb_running_get_entry(args->dnode, NULL, true);
yang_dnode_get_prefix(&prefix, args->dnode, "prefix"); yang_dnode_get_prefix(&prefix, args->dnode, "prefix");
algorithm = yang_dnode_get_uint32(args->dnode, "algo"); algorithm = yang_dnode_get_uint8(args->dnode, "algo");
pcfg = isis_sr_cfg_prefix_add(area, &prefix, algorithm); pcfg = isis_sr_cfg_prefix_add(area, &prefix, algorithm);
pcfg->algorithm = algorithm; pcfg->algorithm = algorithm;

2
yang/frr-isisd.yang
View file

@ -1922,7 +1922,7 @@ module frr-isisd {
if set to disable, ISISEXPLICITNULLFlag if set to disable, ISISEXPLICITNULLFlag
will override the value of ISISPHPFlag"; will override the value of ISISPHPFlag";
leaf algo { leaf algo {
type uint32 { type uint8 {
range "128..255"; range "128..255";
} }
description description