Attention is currently required from: dexter.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/33766
)
Change subject: paging: also accept IMSI strings shorter than 3
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
I think handling the case strlen(imsi)==0 is enough, other cases should contain full imsi
(or at least 3-char prefix).
Also bear in mind this commit simply fixes the bug. Ideally, the behavior would be to
submit the Ass on AGCH if no IMSI is provided, since using prgroup in that case is not
really the good thing to do.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bts/+/33766
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Iab1c3f1c39dd59bb53aa74b2c9e9e135e3985e0b
Gerrit-Change-Number: 33766
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 17 Jul 2023 10:30:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment