Change in ...osmo_gsup[master]: fix E-Routing-Error: both session state / ID IEs are optional

[laforge]

laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo_gsup/+/17864 )

Change subject: fix E-Routing-Error: both session state / ID IEs are optional
......................................................................

fix E-Routing-Error: both session state / ID IEs are optional

If a message that has triggered the E-Routing-Error contains no
session related IEs, they will also be absent in the error message.

Change-Id: I0c0c84c9dbd2e53e7d1720868cc157f9e21749c2
---
M include/gsup_protocol.hrl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/include/gsup_protocol.hrl b/include/gsup_protocol.hrl
index dc49723..2e9c9d3 100644
--- a/include/gsup_protocol.hrl
+++ b/include/gsup_protocol.hrl
@@ -219,7 +219,7 @@
   16#44 => #{message_type => e_forward_access_signalling_req, mandatory => [message_class, source_name, destination_name, an_apdu, session_id, session_state]},
   16#47 => #{message_type => e_close, mandatory => [message_class, source_name, destination_name, session_id, session_state]},
   16#4b => #{message_type => e_abort, mandatory => [message_class, session_id, session_state, bssap_cause]},
-  16#4e => #{message_type => e_routing_err, mandatory => [message_class, source_name, destination_name, session_id, session_state]}
+  16#4e => #{message_type => e_routing_err, mandatory => [message_class, source_name, destination_name], optional => [session_id, session_state]}
 }).
 
 -define(AUTH_TUPLE_MANDATORY, [rand, sres, kc]).

-- 
To view, visit https://gerrit.osmocom.org/c/erlang/osmo_gsup/+/17864
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo_gsup
Gerrit-Branch: master
Gerrit-Change-Id: I0c0c84c9dbd2e53e7d1720868cc157f9e21749c2
Gerrit-Change-Number: 17864
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200419/acccaa16/attachment.htm>