laforge has uploaded this change for review.

View Change

mobile: Fix gsm_refer_pcs(): PCS can only be ARFCN 512..810

While it is correct to use the band indicator from SI1 rest octets,
it may only be applied for ARFCN values in the range 512..810.

Change-Id: Id99c8534bf853f4f24f99364790c1ac1df6cc007
Related: OS#6078
---
M src/host/layer23/src/common/sysinfo.c
1 file changed, 18 insertions(+), 2 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/32/33532/1

diff --git a/src/host/layer23/src/common/sysinfo.c b/src/host/layer23/src/common/sysinfo.c
index 700accd..e3de81f 100644
--- a/src/host/layer23/src/common/sysinfo.c
+++ b/src/host/layer23/src/common/sysinfo.c
@@ -61,8 +61,11 @@
 	if (!s->si1)
 		return 0;
 
-	/* If band indicator indicates PCS band, the cell refers to PCSThe  */
-	return s->band_ind;
+	/* If band indicator indicates PCS band, the cell refers to PCS  */
+	if (s->band_ind && (arfcn >= 512 && arfcn <= 819))
+		return 1;
+
+	return 0;
 }
 
 int gsm48_sysinfo_dump(const struct gsm48_sysinfo *s, uint16_t arfcn,

To view, visit change 33532. To unsubscribe, or for help writing mail filters, visit settings.