laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
HNBAP: Send HNB-REGISTER-REJ on ASN.1 decoding error

Change-Id: Ic4a40966194a57cccc0eb056233f7e7426d6e8f9
---
M src/osmo-hnbgw/hnbgw_hnbap.c
1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/src/osmo-hnbgw/hnbgw_hnbap.c b/src/osmo-hnbgw/hnbgw_hnbap.c
index 5cca3cb..9ffb0ba 100644
--- a/src/osmo-hnbgw/hnbgw_hnbap.c
+++ b/src/osmo-hnbgw/hnbgw_hnbap.c
@@ -422,7 +422,9 @@
rc = hnbap_decode_hnbregisterrequesties(&ies, in);
if (rc < 0) {
LOGHNB(ctx, DHNBAP, LOGL_ERROR, "Failure to decode HNB-REGISTER-REQ: rc=%d\n", rc);
- return rc;
+ cause.present = HNBAP_Cause_PR_protocol;
+ cause.choice.radioNetwork = HNBAP_CauseProtocol_unspecified;
+ return hnbgw_tx_hnb_register_rej(ctx, &cause);
}
asn1_strncpy(identity_str, &ies.hnB_Identity.hNB_Identity_Info, sizeof(identity_str));


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

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ic4a40966194a57cccc0eb056233f7e7426d6e8f9
Gerrit-Change-Number: 36476
Gerrit-PatchSet: 2
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