diff options
| author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-06-03 02:51:12 +0200 |
|---|---|---|
| committer | fixeria <vyanitskiy@sysmocom.de> | 2021-06-09 20:20:42 +0000 |
| commit | 43adbe200c1e1184b444ba8fc16b8bb0c6d8050a (patch) | |
| tree | 4ba41456d4d2748cfd18c914a966512566871c2a | |
| parent | fc8c9880d97636a4bec3b50feb1b8c0c05a49ef8 (diff) | |
measurement: remove over-defensive checks in is_meas_complete()
The timeslot number ts->nr is set in gsm_bts_trx_alloc() and can
never be greater than 7. Regarding the physical channel type
returned by ts_pchan(), it's also unlikely to be greater than
_GSM_PCHAN_MAX.
Change-Id: I6d20d7cba49cc8b6d1dc2192598ca372b7d2c5bf
| -rw-r--r-- | src/common/measurement.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/common/measurement.c b/src/common/measurement.c index 03afaa3e..b5869872 100644 --- a/src/common/measurement.c +++ b/src/common/measurement.c @@ -250,11 +250,6 @@ int is_meas_complete(struct gsm_lchan *lchan, uint32_t fn) int rc = 0; enum gsm_phys_chan_config pchan = ts_pchan(lchan->ts); - if (lchan->ts->nr >= 8) - return -EINVAL; - if (pchan >= _GSM_PCHAN_MAX) - return -EINVAL; - switch (pchan) { case GSM_PCHAN_TCH_F: fn_mod = translate_tch_meas_rep_fn104(fn % 104); |