Change in osmo-hlr[master]: USSD: fix routing to multiple MSC
gerrit-no-reply at lists.osmocom.org
Mon Apr 8 11:58:24 UTC 2019
osmith has posted comments on this change. ( https://gerrit.osmocom.org/13479 )
Change subject: USSD: fix routing to multiple MSC
Patch Set 6:
Thanks for the reviews, here's the next version.
PS5, Line 175:
> let's take this ss->subscr from the top. […]
Updated the patch to have ss->vlr_number, and look it up once if it is empty (like the MT USSD case you have described). The timer is dropped again.
The MO USSD case depends on the libosmocore.git MSC HO GSUP messages patch (needs gsup->source_name), so I've kept it as follow up patch here:
(I've used vlr_number instead of vlr_name, because it is also called vlr_number in hlr_subscriber->vlr_number, and so we don't introduce yet another name for the same thing.)
PS5, Line 251: OGP(DLGSUP, LOG
PS5, Line 260:
> same here.
PS5, Line 263:
> (ack, I have no good idea about naming here)
Changed to "SS/USSD".
PS5, Line 343: ss_tx_error(ss, true, GSM0480_ERR_CODE_SYSTEM_FAILURE);
> (setting ss->subscr to point at local struct subscr becomes invalid memory as soon as the function e […]
Now I get it - it worked in your patch (#4), because you had set it to NULL afterwards. Thanks for explaining.
To view, visit https://gerrit.osmocom.org/13479
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Comment-Date: Mon, 08 Apr 2019 11:58:24 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log