Change in osmo-msc[master]: vlr: optionally send IMEI early to HLR
gerrit-no-reply at lists.osmocom.org
Wed May 8 06:46:48 UTC 2019
Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/13894 )
Change subject: vlr: optionally send IMEI early to HLR
Patch Set 1: Code-Review+1
I would have simply moved the IMEI CHECK unconditionally to the start, but maybe that violates some GSM MAP related spec?
Either way, I'm happy with this approach, aside from the minor comments below.
PS1, Line 955: event without effect
> I guess if this happens, then we have a bug somewhere? I would rather OSMO_ASSERT(0) here... […]
There's two cases:
1) the event is permitted in in_event_mask of the state, but we don't handle it here. Then it's a bug of the code and the code is incomplete, hence we normally put a "default: OSMO_ASSERT(0)" here
2) the event is not permitted in in_event_mask of the state. In this case, the osmo_fsm core will already print a message about an 'unhandled event' and discard the event.
So in either case, the logging here is not needed/intended.
To view, visit https://gerrit.osmocom.org/13894
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-CC: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Comment-Date: Wed, 08 May 2019 06:46:48 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log