summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHolger Hans Peter Freyther <holger@moiji-mobile.com>2017-11-30 16:51:58 +0800
committerHolger Hans Peter Freyther <holger@moiji-mobile.com>2017-11-30 17:03:25 +0800
commit517bda18b21b5127edb92ef79119c885c3b2cd67 (patch)
treec35fcc8c38514f8d49b51711e5732471d2fe3fc8
parent89009751ea7ceda542c254633f4ceef23b3f8f90 (diff)
mobile: Speculative crash fix of the SI pointer
The SIs are kept per ARFCN and for the current cell the cs->si alias will be assigned[1]. On mobile_exit all SIs will be freed but the alias will not be set to NULL. This is a speculative fix but it doesn't seem to make things worse. Related: OS#2690 [1] cs->si = cs->list[cs->arfci].sysinfo; Change-Id: Icf20f9aa03dd26d4bee78772b7f3da034bb34b99
-rw-r--r--src/host/layer23/src/mobile/gsm322.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/host/layer23/src/mobile/gsm322.c b/src/host/layer23/src/mobile/gsm322.c
index 6de8bec..c3485b6 100644
--- a/src/host/layer23/src/mobile/gsm322.c
+++ b/src/host/layer23/src/mobile/gsm322.c
@@ -5141,6 +5141,7 @@ int gsm322_exit(struct osmocom_ms *ms)
}
cs->list[i].flags = 0;
}
+ cs->si = NULL;
/* store BA list */
ba_filename = talloc_asprintf(ms, "%s/%s.ba", config_dir, ms->name);