fixeria submitted this change.

View Change

Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve
s1ap_proxy: handle_ies(): also handle #'ProtocolExtensionField'

This patch prepares for a follow-up change adding E-RAB MODIFY REQ/RSP,
allowing to execute handle_ies() on a list of #'ProtocolExtensionField'.

* 9.1.3.3 E-RAB MODIFY REQUEST
** contains an E-RABToBeModifiedListBearerModReq IE
*** contains E-RABToBeModifiedItemBearerModReq IEs
**** each may optionally contain TransportInformation IE
^^^^^^^^^^^^^^^^^^^^^^^
#'ProtocolExtensionField'

Change-Id: Ic1ed2c4662e82bd739c35f9179eaaec20c3cad98
---
M src/s1ap_proxy.erl
1 file changed, 8 insertions(+), 0 deletions(-)

diff --git a/src/s1ap_proxy.erl b/src/s1ap_proxy.erl
index 18b054b..3414e79 100644
--- a/src/s1ap_proxy.erl
+++ b/src/s1ap_proxy.erl
@@ -592,6 +592,14 @@
{{error, Reason}, S1} ->
{{error, Reason}, S1}
end;
+ #'ProtocolExtensionField'{id = IEI, extensionValue = C0} ->
+ case handle_ie(P, C0, S0) of
+ {{ok, C1}, S1} ->
+ NewIE = IE#'ProtocolExtensionField'{extensionValue = C1},
+ handle_ies([NewIE | Acc], IEI, IEs, S1);
+ {{error, Reason}, S1} ->
+ {{error, Reason}, S1}
+ end;
#'ProtocolIE-Field'{id = ?'id-MME-UE-S1AP-ID', value = Id} ->
S1 = S0#proxy_state{mme_ue_id = Id},
handle_ies([IE | Acc], IEI, IEs, S1);

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

Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ic1ed2c4662e82bd739c35f9179eaaec20c3cad98
Gerrit-Change-Number: 39114
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>