aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVadim Yanitskiy <vyanitskiy@sysmocom.de>2021-07-03 02:42:45 +0200
committerlaforge <laforge@osmocom.org>2021-07-05 12:09:07 +0000
commitfc63164bac8d2782a26b926722492f524b595b38 (patch)
tree8f69ad946b25d9ccd49e6ba8bcd8499ef0d8882b
parente9141c22b109159165dd378f8007fd41ac91ccd8 (diff)
library/RSL_Types: use OCT4 for local/remote address fields
This way functions like f_inet_addr() and f_inet6_addr() can be used directly without converting between bytes and integers. Change-Id: I389a8cb95c025c9dddc751789223621c15d9b48f
-rw-r--r--bsc/BSC_Tests_LCLS.ttcn6
-rw-r--r--bsc/BSC_Tests_VAMOS.ttcn12
-rw-r--r--bsc/MSC_ConnectionHandler.ttcn12
-rw-r--r--bts/BTS_Tests.ttcn6
-rw-r--r--library/RSL_Types.ttcn16
5 files changed, 26 insertions, 26 deletions
diff --git a/bsc/BSC_Tests_LCLS.ttcn b/bsc/BSC_Tests_LCLS.ttcn
index 72fb525b..9771087a 100644
--- a/bsc/BSC_Tests_LCLS.ttcn
+++ b/bsc/BSC_Tests_LCLS.ttcn
@@ -303,7 +303,7 @@ private function f_tc_lcls_ack_rsl_mdcx(RSL_Message rsl_msg, boolean send_on_a)
/* mandatory fields */
fixme_unused := f_rsl_find_ie(rsl_msg, RSL_IE_IPAC_REMOTE_IP, ie);
- host := f_inet_ntoa(int2oct(ie.ipa_remote_ip, 4));
+ host := f_inet_ntoa(ie.ipa_remote_ip);
fixme_unused := f_rsl_find_ie(rsl_msg, RSL_IE_IPAC_REMOTE_PORT, ie);
port_num := ie.ipa_remote_port;
@@ -315,9 +315,9 @@ private function f_tc_lcls_ack_rsl_mdcx(RSL_Message rsl_msg, boolean send_on_a)
}
if (send_on_a == true) {
- CONN_A.send(ts_RSL_IPA_MDCX_ACK(chan_nr, conn_id, oct2int(f_inet_addr(host)), port_num, rtp_pt));
+ CONN_A.send(ts_RSL_IPA_MDCX_ACK(chan_nr, conn_id, f_inet_addr(host), port_num, rtp_pt));
} else {
- CONN_B.send(ts_RSL_IPA_MDCX_ACK(chan_nr, conn_id, oct2int(f_inet_addr(host)), port_num, rtp_pt));
+ CONN_B.send(ts_RSL_IPA_MDCX_ACK(chan_nr, conn_id, f_inet_addr(host), port_num, rtp_pt));
}
}
diff --git a/bsc/BSC_Tests_VAMOS.ttcn b/bsc/BSC_Tests_VAMOS.ttcn
index 9ecb255a..25dee234 100644
--- a/bsc/BSC_Tests_VAMOS.ttcn
+++ b/bsc/BSC_Tests_VAMOS.ttcn
@@ -425,7 +425,7 @@ private function f_reassign_secondary_to_primary_lchan(RslChannelNr new_chan_nr)
rtp_pt := ie.ipa_rtp_pt;
}
RSL.send(ts_RSL_IPA_CRCX_ACK(new_chan_nr, 123,
- oct2int(f_inet_addr("1.2.3.4")),
+ f_inet_addr("1.2.3.4"),
4321,
rtp_pt));
}
@@ -436,7 +436,7 @@ private function f_reassign_secondary_to_primary_lchan(RslChannelNr new_chan_nr)
/* mandatory */
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_IP, ie);
var HostPort peer;
- peer.host := f_inet_ntoa(int2oct(ie.ipa_remote_ip, 4));
+ peer.host := f_inet_ntoa(ie.ipa_remote_ip);
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_PORT, ie);
peer.port_nr := ie.ipa_remote_port;
var uint7_t rtp_pt := 0;
@@ -445,7 +445,7 @@ private function f_reassign_secondary_to_primary_lchan(RslChannelNr new_chan_nr)
rtp_pt := ie.ipa_rtp_pt;
}
RSL.send(ts_RSL_IPA_MDCX_ACK(new_chan_nr, conn_id,
- oct2int(f_inet_addr(peer.host)),
+ f_inet_addr(peer.host),
peer.port_nr,
rtp_pt));
}
@@ -535,7 +535,7 @@ private function f_est_and_reassign_to_secondary_lchan(RslChannelNr new_chan_nr)
rtp_pt := ie.ipa_rtp_pt;
}
RSL.send(ts_RSL_IPA_CRCX_ACK(new_chan_nr, 123,
- oct2int(f_inet_addr("1.2.3.4")),
+ f_inet_addr("1.2.3.4"),
4321,
rtp_pt));
}
@@ -546,7 +546,7 @@ private function f_est_and_reassign_to_secondary_lchan(RslChannelNr new_chan_nr)
/* mandatory */
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_IP, ie);
var HostPort peer;
- peer.host := f_inet_ntoa(int2oct(ie.ipa_remote_ip, 4));
+ peer.host := f_inet_ntoa(ie.ipa_remote_ip);
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_PORT, ie);
peer.port_nr := ie.ipa_remote_port;
var uint7_t rtp_pt := 0;
@@ -555,7 +555,7 @@ private function f_est_and_reassign_to_secondary_lchan(RslChannelNr new_chan_nr)
rtp_pt := ie.ipa_rtp_pt;
}
RSL.send(ts_RSL_IPA_MDCX_ACK(new_chan_nr, conn_id,
- oct2int(f_inet_addr(peer.host)),
+ f_inet_addr(peer.host),
peer.port_nr,
rtp_pt));
}
diff --git a/bsc/MSC_ConnectionHandler.ttcn b/bsc/MSC_ConnectionHandler.ttcn
index 2d0bd277..3e2e4f72 100644
--- a/bsc/MSC_ConnectionHandler.ttcn
+++ b/bsc/MSC_ConnectionHandler.ttcn
@@ -200,7 +200,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
g_media.bts.rtp_pt := ie.ipa_rtp_pt2;
}
RSL.send(ts_RSL_IPA_CRCX_ACK(g_chan_nr, g_media.bts.conn_id,
- oct2int(f_inet_addr(g_media.bts.bts.host)),
+ f_inet_addr(g_media.bts.bts.host),
g_media.bts.bts.port_nr,
g_media.bts.rtp_pt));
g_media.bts.ipa_crcx_seen := true;
@@ -214,7 +214,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
}
/* mandatory */
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_IP, ie);
- g_media.bts.peer.host := f_inet_ntoa(int2oct(ie.ipa_remote_ip, 4));
+ g_media.bts.peer.host := f_inet_ntoa(ie.ipa_remote_ip);
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_PORT, ie);
g_media.bts.peer.port_nr := ie.ipa_remote_port;
/* optional */
@@ -225,7 +225,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
g_media.bts.rtp_pt := ie.ipa_rtp_pt2;
}
RSL.send(ts_RSL_IPA_MDCX_ACK(g_chan_nr, g_media.bts.conn_id,
- oct2int(f_inet_addr(g_media.bts.peer.host)),
+ f_inet_addr(g_media.bts.peer.host),
g_media.bts.peer.port_nr,
g_media.bts.rtp_pt));
g_media.bts.ipa_mdcx_seen := true;
@@ -242,7 +242,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
g_media.bts1.rtp_pt := ie.ipa_rtp_pt2;
}
RSL1.send(ts_RSL_IPA_CRCX_ACK(g_chan_nr, g_media.bts1.conn_id,
- oct2int(f_inet_addr(g_media.bts1.bts.host)),
+ f_inet_addr(g_media.bts1.bts.host),
g_media.bts1.bts.port_nr,
g_media.bts1.rtp_pt));
g_media.bts1.ipa_crcx_seen := true;
@@ -257,7 +257,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
}
/* mandatory */
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_IP, ie);
- g_media.bts1.peer.host := f_inet_ntoa(int2oct(ie.ipa_remote_ip, 4));
+ g_media.bts1.peer.host := f_inet_ntoa(ie.ipa_remote_ip);
b_unused := f_rsl_find_ie(rsl, RSL_IE_IPAC_REMOTE_PORT, ie);
g_media.bts1.peer.port_nr := ie.ipa_remote_port;
/* optional */
@@ -268,7 +268,7 @@ altstep as_Media_ipacc() runs on MSC_ConnHdlr {
g_media.bts1.rtp_pt := ie.ipa_rtp_pt2;
}
RSL1.send(ts_RSL_IPA_MDCX_ACK(g_chan_nr, g_media.bts1.conn_id,
- oct2int(f_inet_addr(g_media.bts1.peer.host)),
+ f_inet_addr(g_media.bts1.peer.host),
g_media.bts1.peer.port_nr,
g_media.bts1.rtp_pt));
g_media.bts1.ipa_mdcx_seen := true;
diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 06085c64..a7fdf163 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -4611,7 +4611,7 @@ testcase TC_ipa_crcx_twice_not_active() runs on test_CT {
private function f_TC_ipa_crcx_mdcx_dlcx_not_active(charstring id) runs on ConnHdlr {
f_rsl_transceive(ts_RSL_IPA_CRCX(g_chan_nr), tr_RSL_IPA_CRCX_ACK(g_chan_nr, ?, ?, ?),
"IPA CRCX ACK");
- var uint32_t remote_ip := f_rnd_int(c_UINT32_MAX);
+ var OCT4 remote_ip := f_rnd_octstring(4);
var uint16_t remote_port := f_rnd_int(c_UINT16_MAX);
var uint7_t rtp_pt2 := f_rnd_int(127);
var uint16_t fake_conn_id := 23; /* we're too lazy to read it out from the CRCX ACK above */
@@ -4632,7 +4632,7 @@ testcase TC_ipa_crcx_mdcx_dlcx_not_active() runs on test_CT {
private function f_TC_ipa_crcx_mdcx_mdcx_dlcx_not_active(charstring id) runs on ConnHdlr {
f_rsl_transceive(ts_RSL_IPA_CRCX(g_chan_nr), tr_RSL_IPA_CRCX_ACK(g_chan_nr, ?, ?, ?),
"IPA CRCX ACK");
- var uint32_t remote_ip := f_rnd_int(c_UINT32_MAX);
+ var OCT4 remote_ip := f_rnd_octstring(4);
var uint16_t remote_port := f_rnd_int(c_UINT16_MAX);
var uint7_t rtp_pt2 := f_rnd_int(127);
var uint16_t fake_conn_id := 23; /* we're too lazy to read it out from the CRCX ACK above */
@@ -4640,7 +4640,7 @@ private function f_TC_ipa_crcx_mdcx_mdcx_dlcx_not_active(charstring id) runs on
tr_RSL_IPA_MDCX_ACK(g_chan_nr, ?, ?, ?, rtp_pt2),
"IPA MDCX ACK");
/* Second MDCX */
- remote_ip := f_rnd_int(c_UINT32_MAX);
+ remote_ip := f_rnd_octstring(4);
remote_port := f_rnd_int(c_UINT16_MAX);
f_rsl_transceive(ts_RSL_IPA_MDCX(g_chan_nr, fake_conn_id, remote_ip, remote_port, rtp_pt2),
tr_RSL_IPA_MDCX_ACK(g_chan_nr, ?, ?, ?, rtp_pt2),
diff --git a/library/RSL_Types.ttcn b/library/RSL_Types.ttcn
index 4e251896..aa206a39 100644
--- a/library/RSL_Types.ttcn
+++ b/library/RSL_Types.ttcn
@@ -919,8 +919,8 @@ module RSL_Types {
uint16_t ipa_conn_id,
uint16_t ipa_local_port,
uint16_t ipa_remote_port,
- uint32_t ipa_local_ip,
- uint32_t ipa_remote_ip,
+ OCT4 ipa_local_ip,
+ OCT4 ipa_remote_ip,
uint8_t ipa_rtp_pt,
uint8_t ipa_rtp_pt2,
RSL_IE_IPA_ConnectionStats ipa_stats,
@@ -2005,7 +2005,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
}
template (value) RSL_Message
ts_RSL_IPA_CRCX(template (value) RslChannelNr chan_nr,
- template (value) uint32_t remote_ip := oct2int('7F000001'O),
+ template (value) OCT4 remote_ip := '7F000001'O,
template (value) uint16_t remote_port := 6766) := {
msg_disc := ts_RSL_MsgDisc(RSL_MDISC_IPACCESS, false),
msg_type := RSL_MT_IPAC_CRCX,
@@ -2018,7 +2018,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
template (value) RSL_Message ts_RSL_IPA_CRCX_ACK(template (value) RslChannelNr chan_nr,
- uint16_t ipa_conn_id, uint32_t local_ip,
+ uint16_t ipa_conn_id, OCT4 local_ip,
uint16_t local_port, uint7_t rtp_pt2) := {
msg_disc := ts_RSL_MsgDisc(RSL_MDISC_IPACCESS, false),
msg_type := RSL_MT_IPAC_CRCX_ACK,
@@ -2032,7 +2032,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
}
template RSL_Message tr_RSL_IPA_CRCX_ACK(template RslChannelNr chan_nr,
template uint16_t ipa_conn_id,
- template uint32_t local_ip,
+ template OCT4 local_ip,
template uint16_t local_port) := {
msg_disc := tr_RSL_MsgDisc(RSL_MDISC_IPACCESS, false),
msg_type := RSL_MT_IPAC_CRCX_ACK,
@@ -2066,7 +2066,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
template (value) RSL_Message ts_RSL_IPA_MDCX(template (value) RslChannelNr chan_nr,
uint16_t ipa_conn_id,
- uint32_t remote_ip, uint16_t remote_port,
+ OCT4 remote_ip, uint16_t remote_port,
uint7_t rtp_pt2) := {
msg_disc := ts_RSL_MsgDisc(RSL_MDISC_IPACCESS, false),
msg_type := RSL_MT_IPAC_MDCX,
@@ -2092,7 +2092,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
template (value) RSL_Message ts_RSL_IPA_MDCX_ACK(template (value) RslChannelNr chan_nr,
uint16_t ipa_conn_id,
- uint32_t local_ip, uint16_t local_port,
+ OCT4 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,
@@ -2108,7 +2108,7 @@ template RSL_Message tr_RSL_MsgTypeDR(template RSL_MessageType msg_type) modifie
}
template RSL_Message tr_RSL_IPA_MDCX_ACK(template RslChannelNr chan_nr,
template uint16_t ipa_conn_id,
- template uint32_t local_ip,
+ template OCT4 local_ip,
template uint16_t local_port,
template uint7_t rtp_pt2) := {
msg_disc := tr_RSL_MsgDisc(RSL_MDISC_IPACCESS, false),