diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-10-25 19:01:50 +0200 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-10-25 19:17:51 +0200 |
| commit | 7be36cfde6a728857c6ec77e12aa6125e8209b5d (patch) | |
| tree | b5d963db28870dc1ac80e5cc338f32192379af4d | |
| parent | f147b5f617cbc64b47453b01419a4a568e007a64 (diff) | |
Set subslots_per_pchan_vamos[GSM_PCHAN_OSMO_DYN] = 0
VAMOS secondary lchans are to be used specifically when the osmocom dyn
TS is set to pchan_is=TCH_{F,H}. Setting secondary subslots for
OSMO_DYN TS is not needed since it's only used to initialize the TS, and
OSMO DYN already initializes 8 subslots
(subslots_per_pchan[GSM_PCHAN_OSMO_DYN]=8). Otherwise, ts_setup_lchans()
will try to initialize 8+2 lchans on the TS, which is more than needed
and will access out of bounds in the array.
Related: OS#5278
Change-Id: I8727d5b446179c0ebcd8738507efe5a50afaf1e2
| -rw-r--r-- | src/osmo-bsc/gsm_data.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/osmo-bsc/gsm_data.c b/src/osmo-bsc/gsm_data.c index b0e0a68d1..0134e34c9 100644 --- a/src/osmo-bsc/gsm_data.c +++ b/src/osmo-bsc/gsm_data.c @@ -633,7 +633,7 @@ static const uint8_t subslots_per_pchan_vamos[] = { [GSM_PCHAN_SDCCH8_SACCH8C] = 0, [GSM_PCHAN_CCCH_SDCCH4_CBCH] = 0, [GSM_PCHAN_SDCCH8_SACCH8C_CBCH] = 0, - [GSM_PCHAN_OSMO_DYN] = 2, + [GSM_PCHAN_OSMO_DYN] = 0, [GSM_PCHAN_TCH_F_PDCH] = 2, }; |