aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPhilipp <pmaier@sysmocom.de>2016-11-15 19:27:20 +0100
committerHarald Welte <laforge@gnumonks.org>2016-11-16 16:35:42 +0000
commit38cba5aa7972260c4444ecea3fd47651a5d54059 (patch)
tree2d79f3a33df3da3c33146b4f96b6275dc7c922c8
parentfb89b9b00b6be88d3822fc206f117660e038a046 (diff)
OM2000: Fixup based on Coverity Scan suggestion
This commit fixes Coverity Scan defect: CID 151901: Insecure data handling (TAINTED_SCALAR) Passing tainted variable "tag_len" to a tainted sink. Change-Id: Ic71ed6a3bbb228bc03e95bfc4a6f5fe09cf5a021
-rw-r--r--openbsc/src/libbsc/abis_om2000.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/openbsc/src/libbsc/abis_om2000.c b/openbsc/src/libbsc/abis_om2000.c
index 2733a90fd..ae23e507d 100644
--- a/openbsc/src/libbsc/abis_om2000.c
+++ b/openbsc/src/libbsc/abis_om2000.c
@@ -2438,10 +2438,10 @@ static int process_mo_state(struct gsm_bts *bts, struct om2k_decoded_msg *odm)
}
/* Display fault report bits (helper function of display_fault_maps()) */
-static bool display_fault_bits(const uint8_t *vect, unsigned int len,
+static bool display_fault_bits(const uint8_t *vect, uint16_t len,
uint8_t dei, const struct abis_om2k_mo *mo)
{
- int i;
+ uint16_t i;
int k;
bool faults_present = false;
int first = 1;