Change in osmo-ttcn3-hacks[master]: MGCP_Test: Make sure PT in RTP-Stream and MGCP match up

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Sat Jul 21 08:21:00 UTC 2018


Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/10076 )

Change subject: MGCP_Test: Make sure PT in RTP-Stream and MGCP match up
......................................................................

MGCP_Test: Make sure PT in RTP-Stream and MGCP match up

At the moment the RTP stream emulation is left in its default
configuration, this means that the payload type that appears in the RTP
stream is always 0. This may mismatch the payload type that is
configured with MGCP.

If nothing else is set, we should make sure that whan we create and modify
flows, the RTP emulation is always reconfigured to use the payload-type that
we set in the flow parameters. The other rtp-emulation parameters should
be set to their defaults.

- Make sure f_flow_modify and f_flow_create set the rtp flow parameters
  properly.

Change-Id: Ie888424ac3e0bf0d960b6f071855b6dd43935a0e
Related:OS#3384
---
M mgw/MGCP_Test.ttcn
1 file changed, 21 insertions(+), 1 deletion(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn
index d816a71..659d5f1 100644
--- a/mgw/MGCP_Test.ttcn
+++ b/mgw/MGCP_Test.ttcn
@@ -58,6 +58,7 @@
 	private function f_init(template MgcpEndpoint ep := omit) runs on dummy_CT {
 		var Result res;
 		var uint32_t ssrc;
+
 		if (initialized == false) {
 			initialized := true;
 
@@ -243,7 +244,8 @@
 		HostPort mgw,	/* mgw side */
 		uint7_t pt,
 		charstring codec,
-		MgcpConnectionId mgcp_conn_id optional
+		MgcpConnectionId mgcp_conn_id optional,
+		RtpemConfig rtp_cfg optional
 	}
 
 	/* Create an RTP flow (bidirectional, or receive-only) */
@@ -256,6 +258,15 @@
 		/* bind local RTP emulation socket */
 		f_rtpem_bind(pt, flow.em.hostname, flow.em.portnr);
 
+		/* configure rtp-emulation */
+		if (ispresent(flow.rtp_cfg)) {
+			f_rtpem_configure(pt, flow.rtp_cfg);
+		} else {
+			var RtpemConfig rtp_cfg := c_RtpemDefaultCfg;
+			rtp_cfg.tx_payload_type := flow.pt
+			f_rtpem_configure(pt, rtp_cfg);
+		}
+
 		if (one_phase) {
 			/* Connect flow to MGW using a CRCX that also contains an SDP
 			 * part that tells the MGW where we are listening for RTP streams
@@ -299,6 +310,15 @@
 		/* rebind local RTP emulation socket to the new address */
 		f_rtpem_bind(pt, flow.em.hostname, flow.em.portnr);
 
+		/* reconfigure rtp-emulation */
+		if (ispresent(flow.rtp_cfg)) {
+			f_rtpem_configure(pt, flow.rtp_cfg);
+		} else {
+			var RtpemConfig rtp_cfg := c_RtpemDefaultCfg;
+			rtp_cfg.tx_payload_type := flow.pt
+			f_rtpem_configure(pt, rtp_cfg);
+		}
+
 		/* connect MGW side RTP socket to the emulation-side RTP socket using SDP */
 		cmd := ts_MDCX(get_next_trans_id(), ep, mode, call_id, flow.mgcp_conn_id);
 		cmd.sdp := ts_SDP(flow.em.hostname, flow.em.hostname, "23", "42",

-- 
To view, visit https://gerrit.osmocom.org/10076
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie888424ac3e0bf0d960b6f071855b6dd43935a0e
Gerrit-Change-Number: 10076
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180721/1b95d752/attachment.htm>


More information about the gerrit-log mailing list