cellular-infrastructure
/
osmo-pcu
mirror of https://gerrit.osmocom.org/osmo-pcu
9
0
Fork 1
Code Issues Releases Wiki Activity

vty: Fix value range of commands accepting csecs 

Currently an uint8_t csec variable is used to temporarily store
the number of centiseconds which can be set within the range
1 - 65535. This way values above 255 cannot be set properly. This
affects the VTY commands "queue lifetime", "queue hysteres", and
"queue idle-ack-delay".

This commit replaces the uint8_t by an uint16_t.

Sponsored-by: On-Waves ehf
This commit is contained in:
Jacob Erlbeck 2015-04-30 17:55:16 +02:00
parent c0c580c414
commit 2acfbebfd3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/pcu_vty.c
View File

@ -152,7 +152,7 @@ DEFUN(cfg_pcu_queue_lifetime,
QUEUE_STR LIFETIME_STR "Lifetime in centi-seconds")
{
struct gprs_rlcmac_bts *bts = bts_main_data();
uint8_t csec = atoi(argv[0]);
uint16_t csec = atoi(argv[0]);
bts->force_llc_lifetime = csec;
@ -193,7 +193,7 @@ DEFUN(cfg_pcu_queue_hysteresis,
QUEUE_STR QUEUE_HYSTERESIS_STR "Hysteresis in centi-seconds")
{
struct gprs_rlcmac_bts *bts = bts_main_data();
uint8_t csec = atoi(argv[0]);
uint16_t csec = atoi(argv[0]);
bts->llc_discard_csec = csec;
@ -220,7 +220,7 @@ DEFUN(cfg_pcu_queue_idle_ack_delay,
QUEUE_STR QUEUE_IDLE_ACK_STR "Idle ACK delay in centi-seconds")
{
struct gprs_rlcmac_bts *bts = bts_main_data();
uint8_t csec = atoi(argv[0]);
uint16_t csec = atoi(argv[0]);
bts->llc_idle_ack_csec = csec;