Harald Welte has submitted this change and it was merged. Change subject: RSL: ts_RSL_IPA_MDCX_ACK contains *local* IP+Port, not remote ......................................................................
RSL: ts_RSL_IPA_MDCX_ACK contains *local* IP+Port, not remote the IPA_MDCX_ACK contains the BTS-side IP/Port, despite the MDCX containing the BSC-side IP/Port! Change-Id: Ided8acff720aeb52232d17c8f142a97557f4b424 --- M library/RSL_Types.ttcn 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/library/RSL_Types.ttcn b/library/RSL_Types.ttcn index 45d6aec..ddee44d 100644 --- a/library/RSL_Types.ttcn +++ b/library/RSL_Types.ttcn @@ -1369,7 +1369,7 @@ } template RSL_Message ts_RSL_IPA_MDCX_ACK(RslChannelNr chan_nr, template uint16_t ipa_conn_id, - uint32_t remote_ip, uint16_t remote_port, + uint32_t local_ip, uint16_t local_port, uint7_t rtp_pt2) := { msg_disc := ts_RSL_MsgDisc(RSL_MDISC_IPACCESS, false), msg_type := RSL_MT_IPAC_MDCX_ACK, @@ -1377,8 +1377,8 @@ t_RSL_IE(RSL_IE_CHAN_NR, RSL_IE_Body:{chan_nr := chan_nr}), /* optional */ t_RSL_IE(RSL_IE_IPAC_CONN_ID, RSL_IE_Body:{ipa_conn_id := ipa_conn_id}), - t_RSL_IE(RSL_IE_IPAC_REMOTE_IP, RSL_IE_Body:{ipa_remote_ip := remote_ip}), - t_RSL_IE(RSL_IE_IPAC_REMOTE_PORT, RSL_IE_Body:{ipa_remote_port := remote_port}), + t_RSL_IE(RSL_IE_IPAC_LOCAL_IP, RSL_IE_Body:{ipa_local_ip := local_ip}), + t_RSL_IE(RSL_IE_IPAC_LOCAL_PORT, RSL_IE_Body:{ipa_local_port := local_port}), /* optional: RTP Payload Type */ t_RSL_IE(RSL_IE_IPAC_RTP_PAYLOAD2, RSL_IE_Body:{ipa_rtp_pt2 := rtp_pt2}) } -- To view, visit https://gerrit.osmocom.org/6892 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ided8acff720aeb52232d17c8f142a97557f4b424 Gerrit-PatchSet: 2 Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Owner: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder