laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-hnbgw/+/36476?usp=email )

Change subject: HNBAP: Send HNB-REGISTER-REJ on ASN.1 decoding error
......................................................................

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(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved




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 https://gerrit.osmocom.org/c/osmo-hnbgw/+/36476?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ic4a40966194a57cccc0eb056233f7e7426d6e8f9
Gerrit-Change-Number: 36476
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to