Review at  https://gerrit.osmocom.org/3262

gsm_bts_trx_set_system_infos(): Disable non-existing SI

If we previously had a given SI present/active, we must send a
zero-length BCCH FILLING for that SI type to the BTS to stop it from
further transmitting this SI.

Change-Id: I33e356e2fa3a69efac9080813e3e9ef4e6438ed1
Closes: OS#2368
---
M openbsc/src/libbsc/bsc_init.c
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/62/3262/1

diff --git a/openbsc/src/libbsc/bsc_init.c b/openbsc/src/libbsc/bsc_init.c
index c17e4a7..20d82a9 100644
--- a/openbsc/src/libbsc/bsc_init.c
+++ b/openbsc/src/libbsc/bsc_init.c
@@ -192,9 +192,13 @@
 
        for (n = 0; n < n_si; n++) {
                i = gen_si[n];
+               /* if we don't currently have this SI, we send a zero-length
+                * RSL BCCH FILLING / SACCH FILLING * in order to deactivate
+                * the SI, in case it might have previously been active */
                if (!GSM_BTS_HAS_SI(bts, i))
-                       continue;
-               rc = rsl_si(trx, i, si_len[i]);
+                       rc = rsl_si(trx, i, 0);
+               else
+                       rc = rsl_si(trx, i, si_len[i]);
                if (rc < 0)
                        return rc;
        }

-- 
To view, visit https://gerrit.osmocom.org/3262
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I33e356e2fa3a69efac9080813e3e9ef4e6438ed1
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <[email protected]>

Reply via email to