ttcn3
/
osmo-ttcn3-hacks
mirror of https://gerrit.osmocom.org/osmo-ttcn3-hacks
10
0
Fork 1
Code Issues Releases Wiki Activity

mgw: Fix logic in test TC_two_crcx_mdcx_and_rtp_osmux 

rtpem should be set to BIDIR at the same time where MDCX "sendrecv" is
sent, otherwise if MGW sends us RTP packets (because we set the conn to
sendrecv) they will be counted incorrectly in
stats[0].num_pkts_rx_err_disabled.

This issue doesn't trigger in current code because the MGW doesn't know
anyway the remote IP address of the other connection until an MDCX is
sent to it. However, if for whatever reason the IP address is known (for
instance because it is set during CRCX, which will be done in next
commits), then RTP messages would be sent and the error counter would
be > 0.

Change-Id: I8f2c4e497e522fc17e5cfe76987f802265c486ab
This commit is contained in:
Pau Espin 2020-09-08 12:16:17 +02:00 committed by pespin
parent 89c76d2102
commit e67b1c64ce
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
mgw/MGCP_Test.ttcn
View File

@ -1331,6 +1331,9 @@ module MGCP_Test {
/* The first leg will now be switched into bidirectional
* mode, but we do not expect any data comming back yet. */
f_rtpem_mode(RTPEM[0], RTPEM_MODE_BIDIR);
stats_osmux := f_osmuxem_stats_get(OsmuxEM);
num_pkts_tx[1] := stats_osmux.num_pkts_tx;
f_flow_modify(RTPEM[0], ep, call_id, "sendrecv", flow[0]);
f_sleep(0.5);
stats_rtp := f_rtpem_stats_get(RTPEM[0]);
@ -1347,12 +1350,9 @@ module MGCP_Test {
/* When the second leg is switched into bidirectional mode
* as well, then the MGW will connect the two together and
* we should see RTP streams passing through from both ends. */
f_rtpem_mode(RTPEM[0], RTPEM_MODE_BIDIR);
f_osmuxem_mode(OsmuxEM, OSMUXEM_MODE_BIDIR);
stats_rtp := f_rtpem_stats_get(RTPEM[0]);
num_pkts_tx[0] := stats_rtp.num_pkts_tx
stats_osmux := f_osmuxem_stats_get(OsmuxEM);
num_pkts_tx[1] := stats_osmux.num_pkts_tx
num_pkts_tx[0] := stats_rtp.num_pkts_tx;
if (crcx_osmux_wildcard) {
/* For now we must set same CID as the MGW recvCID,