<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24250">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">MGCP_Test: fix TC_one_crcx_loopback_rtp_implicit<br><br>The testcase TC_one_crcx_loopback_rtp_implicit uses<br>f_TC_one_crcx_loopback_rtp, which creates the RTP flow with IPv4<br>addresses but since we do not send a local RTP IP address with the CRCX<br>to the MGW, the MGW will prefer IPv6, which means that we get an IPv6<br>address back while the RTP strem is IPv4 on the TTCN3 side.<br><br>Related: OS#5123<br>Change-Id: I80498737d5b32f28b62e0c17cce1969b54af948c<br>---<br>M mgw/MGCP_Test.ttcn<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn</span><br><span>index c4f97a8..5575f7a 100644</span><br><span>--- a/mgw/MGCP_Test.ttcn</span><br><span>+++ b/mgw/MGCP_Test.ttcn</span><br><span>@@ -1693,14 +1693,14 @@</span><br><span> </span><br><span>     /* Create one connection in loopback mode, test if the RTP packets are</span><br><span>        * actually reflected */</span><br><span style="color: hsl(0, 100%, 40%);">-        function f_TC_one_crcx_loopback_rtp(boolean one_phase := true) runs on dummy_CT {</span><br><span style="color: hsl(120, 100%, 40%);">+     function f_TC_one_crcx_loopback_rtp(charstring local_ip, charstring remote_ip, boolean one_phase := true) runs on dummy_CT {</span><br><span>                 var RtpFlowData flow;</span><br><span>                var MgcpEndpoint ep := c_mgw_ep_rtpbridge & "1@" & c_mgw_domain;</span><br><span>           var MgcpCallId call_id := '1225'H;</span><br><span>           var RtpemStats stats;</span><br><span> </span><br><span>            f_init(ep);</span><br><span style="color: hsl(0, 100%, 40%);">-             flow := valueof(t_RtpFlow(mp_local_ipv4, mp_remote_ipv4, 111, "GSM-HR-08/8000/1"));</span><br><span style="color: hsl(120, 100%, 40%);">+         flow := valueof(t_RtpFlow(local_ip, remote_ip, 111, "GSM-HR-08/8000/1"));</span><br><span>          flow.em.portnr := 10000;</span><br><span>             f_flow_create(RTPEM[0], ep, call_id, "loopback", flow, one_phase := one_phase);</span><br><span> </span><br><span>@@ -1725,7 +1725,7 @@</span><br><span>        /* Create one connection in loopback mode, test if the RTP packets are</span><br><span>        * actually reflected */</span><br><span>     testcase TC_one_crcx_loopback_rtp() runs on dummy_CT {</span><br><span style="color: hsl(0, 100%, 40%);">-           f_TC_one_crcx_loopback_rtp(one_phase := true)</span><br><span style="color: hsl(120, 100%, 40%);">+                 f_TC_one_crcx_loopback_rtp(mp_local_ipv4, mp_remote_ipv4, one_phase := true)</span><br><span>        }</span><br><span> </span><br><span>        /* Same as above, but we will intenionally not tell the MGW where to</span><br><span>@@ -1733,7 +1733,7 @@</span><br><span>          * loopback mode, so the MGW should take the originating address from</span><br><span>         * the incoming RTP packet and send it back to the source */</span><br><span>         testcase TC_one_crcx_loopback_rtp_implicit() runs on dummy_CT {</span><br><span style="color: hsl(0, 100%, 40%);">-          f_TC_one_crcx_loopback_rtp(one_phase := false)</span><br><span style="color: hsl(120, 100%, 40%);">+                f_TC_one_crcx_loopback_rtp(mp_local_ipv6, mp_remote_ipv6, one_phase := false)</span><br><span>       }</span><br><span> </span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24250">change 24250</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24250"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I80498737d5b32f28b62e0c17cce1969b54af948c </div>
<div style="display:none"> Gerrit-Change-Number: 24250 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>