pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/39008?usp=email ) ( 3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: vty: Avoid accessing gsupclient object fields directly ...................................................................... vty: Avoid accessing gsupclient object fields directly Use new APIs recently added. Change-Id: I7ead95bb86d1ab510090c29390255007923f895f Depends: osmo-hlr.git Change-Id I401af83232022f1c141eef1f428cbe206a8aaaa2 --- M src/libmsc/e_link.c 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/libmsc/e_link.c b/src/libmsc/e_link.c index 4b61ec7..62cf823 100644 --- a/src/libmsc/e_link.c +++ b/src/libmsc/e_link.c @@ -114,10 +114,13 @@ struct vlr_subscr *vsub; const char *local_msc_name = NULL; - if (e->gcm && e->gcm->gsup_client && e->gcm->gsup_client->ipa_dev) { - local_msc_name = e->gcm->gsup_client->ipa_dev->serno; - if (!local_msc_name) - local_msc_name = e->gcm->gsup_client->ipa_dev->unit_name; + if (e->gcm && e->gcm->gsup_client) { + const struct ipaccess_unit *ipa_dev = osmo_gsup_client_get_ipaccess_unit(e->gcm->gsup_client); + if (ipa_dev) { + local_msc_name = ipa_dev->serno; + if (!local_msc_name) + local_msc_name = ipa_dev->unit_name; + } } if (!local_msc_name) { -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/39008?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I7ead95bb86d1ab510090c29390255007923f895f Gerrit-Change-Number: 39008 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>