diff options
| author | Daniel Willmann <daniel@totalueberwachung.de> | 2011-08-18 15:22:01 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <zecke@selfish.org> | 2012-09-11 16:20:45 +0200 |
| commit | d08ad7d6d9d46fdd1d4565ff4b42201f976dd334 (patch) | |
| tree | 4884d48942fda2d55e4680e7e2554b9594249281 /openbsc | |
| parent | ebe22375a6adcbc65c8d1a9367c5b16efdd6215c (diff) | |
ctrl: Use strtol instead of atoi to detect conversion errors
Diffstat (limited to 'openbsc')
| -rw-r--r-- | openbsc/src/libctrl/control_if.c | 37 |
1 files changed, 27 insertions, 10 deletions
diff --git a/openbsc/src/libctrl/control_if.c b/openbsc/src/libctrl/control_if.c index 98748d585..e76b4e5f1 100644 --- a/openbsc/src/libctrl/control_if.c +++ b/openbsc/src/libctrl/control_if.c @@ -115,10 +115,30 @@ struct ctrl_cmd *ctrl_cmd_trap(struct ctrl_cmd *cmd) return trap; } +static int get_num(vector vline, int i, long *num) +{ + char *token, *tmp; + + if (i >= vector_active(vline)) + return 0; + token = vector_slot(vline, i); + + errno = 0; + if (token[0] == '\0') + return 0; + + *num = strtol(token, &tmp, 10); + if (tmp[0] != '\0' || errno != 0) + return 0; + + return 1; +} + int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data) { char *token, *request; - int num, i, j, ret, node; + long num; + int i, j, ret, node; struct gsm_network *gsmnet = data; struct gsm_network *net = NULL; @@ -163,10 +183,9 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data) if (!net) goto err_missing; i++; - if (i >= vector_active(vline)) + if (!get_num(vline, i, &num)) goto err_index; - token = vector_slot(vline, i); - num = atoi(token); + bts = gsm_bts_num(net, num); if (!bts) goto err_missing; @@ -176,10 +195,9 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data) if (!bts) goto err_missing; i++; - if (i >= vector_active(vline)) + if (!get_num(vline, i, &num)) goto err_index; - token = vector_slot(vline, i); - num = atoi(token); + trx = gsm_bts_trx_num(bts, num); if (!trx) goto err_missing; @@ -189,10 +207,9 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data) if (!trx) goto err_missing; i++; - if (i >= vector_active(vline)) + if (!get_num(vline, i, &num)) goto err_index; - token = vector_slot(vline, i); - num = atoi(token); + if ((num >= 0) && (num < TRX_NR_TS)) ts = &trx->ts[num]; if (!ts) |