diff options
| author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-06-16 18:25:57 +0200 |
|---|---|---|
| committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-06-18 18:56:57 +0200 |
| commit | 64e1c1222e1331536fe45f1505229126e0b25fc9 (patch) | |
| tree | d035d4b28b9ddc4330fe615cb035bca600ed9651 | |
| parent | 4e8d4d5bb87e7fa532901af1da0470b3f1850c66 (diff) | |
oml: fix handling of NM_ATT_INTERF_BOUND attribute
Looks like this part of the code has never been tested. The old
code would dereference the same value in the loop and assign it
to all members in array 'bts->interference.boundary'.
Change-Id: I7f83d8e6eb6cc19e3e9529ba06617a902de23e35
Related: SYS#5313, OS#1569
| -rw-r--r-- | src/common/oml.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/common/oml.c b/src/common/oml.c index 8b84f859..aeded8bf 100644 --- a/src/common/oml.c +++ b/src/common/oml.c @@ -586,7 +586,7 @@ static int oml_rx_set_bts_attr(struct gsm_bts *bts, struct msgb *msg) if (TLVP_PRES_LEN(&tp, NM_ATT_INTERF_BOUND, 6)) { payload = TLVP_VAL(&tp, NM_ATT_INTERF_BOUND); for (i = 0; i < 6; i++) { - int16_t boundary = *payload; + const int16_t boundary = payload[i]; bts->interference.boundary[i] = -1 * boundary; } } |