Attention is currently required from: neels. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/30821 )
Change subject: bsc: Fix crash if PagingResponse with invalid MobileIdentity is received ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
File src/osmo-bsc/gsm_08_08.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/30821/comment/81daa178_b1c7f49d PS1, Line 443: if (conn->bsub)
in general osmo-bsc code assumes that the subscriber is non-NULL, we can work around like this, but […]
We definetly need to find the subscriber since the paging requests are now stored in a list per subscriber (eg if a subscriber is paged for the entire BSS and you have 200 BTS, then you have 200 paging requests per subscriber). Having them reachable per subscriber is hence really saving lots of CPU time.
Hence it really makes no sense to call paging_request_stop() if bsub is NULL now, since anyway nothing would be done in that case.