diff options
author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2017-11-30 16:51:58 +0800 |
---|---|---|
committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2017-11-30 17:03:25 +0800 |
commit | 517bda18b21b5127edb92ef79119c885c3b2cd67 (patch) | |
tree | c35fcc8c38514f8d49b51711e5732471d2fe3fc8 /src/host/layer23/src/mobile | |
parent | 89009751ea7ceda542c254633f4ceef23b3f8f90 (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
Diffstat (limited to 'src/host/layer23/src/mobile')
-rw-r--r-- | src/host/layer23/src/mobile/gsm322.c | 1 |
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 6de8becc..c3485b6a 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); |