laforge submitted this change.

View Change


Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified
HNBAP: Support IMSI identity type in hnbgw_tx_ue_register_rej()

Change-Id: I2e00968cbf686f78f5c9655e899963f2b84dd78b
---
M src/osmo-hnbgw/hnbgw_hnbap.c
1 file changed, 21 insertions(+), 0 deletions(-)

diff --git a/src/osmo-hnbgw/hnbgw_hnbap.c b/src/osmo-hnbgw/hnbgw_hnbap.c
index fc06462..4fa59b9 100644
--- a/src/osmo-hnbgw/hnbgw_hnbap.c
+++ b/src/osmo-hnbgw/hnbgw_hnbap.c
@@ -211,6 +211,7 @@
{
HNBAP_UERegisterReject_t reject_out;
HNBAP_UERegisterRejectIEs_t reject;
+ char imsi[GSM23003_IMSI_MAX_DIGITS+1];
struct msgb *msg;
int rc;

@@ -271,6 +272,14 @@
ue_id->choice.pTMSIRAI.rAI.rAC.size);
break;

+ case HNBAP_UE_Identity_PR_iMSI:
+ ranap_bcd_decode(imsi, sizeof(imsi), ue_id->choice.iMSI.buf, ue_id->choice.iMSI.size);
+ LOGHNB(hnb, DHNBAP, LOGL_DEBUG, "REJ UE_Id IMSI %s\n", imsi);
+
+ OCTET_STRING_fromBuf(&reject.uE_Identity.choice.iMSI,
+ (const char *)ue_id->choice.iMSI.buf, ue_id->choice.iMSI.size);
+ break;
+
default:
LOGHNB(hnb, DHNBAP, LOGL_ERROR, "Cannot compose UE Register Reject:"
" unsupported UE ID (present=%d)\n", ue_id->present);
@@ -312,6 +321,9 @@
ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_OCTET_STRING,
&reject.uE_Identity.choice.pTMSIRAI.rAI.rAC);
break;
+ case HNBAP_UE_Identity_PR_iMSI:
+ ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_OCTET_STRING,
+ &reject.uE_Identity.choice.iMSI);

default:
/* should never happen after above switch() */

To view, visit change 36481. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I2e00968cbf686f78f5c9655e899963f2b84dd78b
Gerrit-Change-Number: 36481
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged