diff options
| author | Harald Welte <laforge@gnumonks.org> | 2017-07-20 01:19:18 +0200 |
|---|---|---|
| committer | Neels Hofmeyr <neels@hofmeyr.de> | 2017-08-27 17:40:54 +0200 |
| commit | 3f263d4cb48f610350f738dcdabb51f7b200dd6b (patch) | |
| tree | 6dc081ff7c93d57326747a355a13d75c6dfa2ed2 /src/libbsc | |
| parent | 72aaf4ce651e80185d6f39f418dba11d9481a542 (diff) | |
bsc_vty: Don't allow timers of zero (0)
It typically doesn't make sense to configure any of the GSM RR timer
to 0 (Seconds). In fact, accidentially configuring any of the timers
to zero might have severe side effects, such as "stuck channels"
described in https://osmocom.org/issues/2380
Change-Id: I517828f2f0c80ec01cb63648db2626f17a67fe57
Diffstat (limited to 'src/libbsc')
| -rw-r--r-- | src/libbsc/bsc_vty.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/libbsc/bsc_vty.c b/src/libbsc/bsc_vty.c index 1e78b08b1..bf7c3cac5 100644 --- a/src/libbsc/bsc_vty.c +++ b/src/libbsc/bsc_vty.c @@ -1543,14 +1543,14 @@ DEFUN(cfg_net_pag_any_tch, #define DECLARE_TIMER(number, doc) \ DEFUN(cfg_net_T##number, \ cfg_net_T##number##_cmd, \ - "timer t" #number " <0-65535>", \ + "timer t" #number " <1-65535>", \ "Configure GSM Timers\n" \ doc "Timer Value in seconds\n") \ { \ struct gsm_network *gsmnet = gsmnet_from_vty(vty); \ int value = atoi(argv[0]); \ \ - if (value < 0 || value > 65535) { \ + if (value < 1 || value > 65535) { \ vty_out(vty, "Timer value %s out of range.%s", \ argv[0], VTY_NEWLINE); \ return CMD_WARNING; \ |