laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/38426?usp=email )
Change subject: ss7 vty: Limit route priority range to 1-9 ......................................................................
ss7 vty: Limit route priority range to 1-9
This matches route priority ranges used by other vendors.
Change-Id: I36ceee5d7cb1b2287c4a2d7ad67f478843300a0b --- M src/osmo_ss7_vty.c M tests/vty/osmo_stp_test.vty 2 files changed, 2 insertions(+), 2 deletions(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, approved
diff --git a/src/osmo_ss7_vty.c b/src/osmo_ss7_vty.c index 356090c..e3142c7 100644 --- a/src/osmo_ss7_vty.c +++ b/src/osmo_ss7_vty.c @@ -74,7 +74,7 @@ QOS_CLASS_RANGE_HELP_STR \ "Default QoS Class (0)\n"
-#define ROUTE_PRIO_RANGE_STR "<0-4294967295>" +#define ROUTE_PRIO_RANGE_STR "<1-9>" #define ROUTE_PRIO_RANGE_HELP_STR "Priority\n" #define ROUTE_PRIO_VAR_STR "(" ROUTE_PRIO_RANGE_STR "|default)" #define ROUTE_PRIO_VAR_HELP_STR \ diff --git a/tests/vty/osmo_stp_test.vty b/tests/vty/osmo_stp_test.vty index 7967216..cb24560 100644 --- a/tests/vty/osmo_stp_test.vty +++ b/tests/vty/osmo_stp_test.vty @@ -570,7 +570,7 @@ OsmoSTP(config-cs7)# route-table system OsmoSTP(config-cs7-rt)# list ... - update route POINT_CODE MASK linkset LS_NAME [priority] [(<0-4294967295>|default)] [qos-class] [(<0-7>|default)] + update route POINT_CODE MASK linkset LS_NAME [priority] [(<1-9>|default)] [qos-class] [(<0-7>|default)] remove route POINT_CODE MASK OsmoSTP(config-cs7-rt)# update route 3.2.1 7.255.7 linkset my-ass OsmoSTP(config-cs7-rt)# show running-config