Change in osmo-hlr[master]: USSD: save MO USSD's originating MSC's vlr_number

osmith gerrit-no-reply at lists.osmocom.org
Tue Apr 9 08:05:22 UTC 2019


osmith has posted comments on this change. ( https://gerrit.osmocom.org/13515 )

Change subject: USSD: save MO USSD's originating MSC's vlr_number
......................................................................


Patch Set 8:

(2 comments)

https://gerrit.osmocom.org/#/c/13515/7/src/hlr_ussd.c
File src/hlr_ussd.c:

https://gerrit.osmocom.org/#/c/13515/7/src/hlr_ussd.c@562
PS7, Line 562: 				if (gsup->source_name) {
> ok so this place is correct, but use conn->where_ever_you_find_the_peers_ipa_name
Done


https://gerrit.osmocom.org/#/c/13515/7/src/hlr_ussd.c@569
PS7, Line 569: 		} else {
TL;DR of handle_ussd() [1] is:

> return -ENOTSUP;

So I think it is fine to only care about ss->vlr_number in the "if (ss_op_is_ussd(req.opcode)) {" case.

[1]: https://git.osmocom.org/osmo-hlr/tree/src/hlr_ussd.c?id=f7d3251d9ac4ba29fc2cebc55106c2400d8a9423#n395



-- 
To view, visit https://gerrit.osmocom.org/13515
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If5a65f471672949192061c5fe396603611123bc1
Gerrit-Change-Number: 13515
Gerrit-PatchSet: 8
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Comment-Date: Tue, 09 Apr 2019 08:05:22 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190409/7b0ab8f5/attachment.html>


More information about the gerrit-log mailing list