Change in osmo-hnodeb[master]: rtp: Use dynamic Payload Type 96

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/.

pespin gerrit-no-reply at lists.osmocom.org
Thu Dec 16 16:39:29 UTC 2021


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hnodeb/+/26596 )

Change subject: rtp: Use dynamic Payload Type 96
......................................................................

rtp: Use dynamic Payload Type 96

Change-Id: I6ca83c274200d8fc0fb3d8914e970b05df91f6ea
---
M src/osmo-hnodeb/rtp.c
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/src/osmo-hnodeb/rtp.c b/src/osmo-hnodeb/rtp.c
index 2f40d99..41fb668 100644
--- a/src/osmo-hnodeb/rtp.c
+++ b/src/osmo-hnodeb/rtp.c
@@ -193,6 +193,14 @@
 		LOGUE(ue, DRTP, LOGL_ERROR, "Failed to set RTP socket parameters: %s\n", strerror(-rc));
 		goto free_ret;
 	}
+	/* TS 25.414 Section 5.1.3.3.1.6: A dynamic Payload Type (IETF RFC 1890
+	 * [23]) shall be used. Values in the Range between 96 and 127 shall be
+	 * used. The value shall be ignored in the receiving entity. */
+	rc = osmo_rtp_socket_set_pt(rs, 96);
+	if (rc < 0) {
+		LOGUE(ue, DRTP, LOGL_ERROR, "Failed to set RTP socket Payload-Type 96\n");
+		/* Continue, the other side is anyway ignoring it... */
+	}
 	rs->priv = conn;
 	rs->rx_cb = &rtp_rx_cb;
 

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-hnodeb/+/26596
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hnodeb
Gerrit-Branch: master
Gerrit-Change-Id: I6ca83c274200d8fc0fb3d8914e970b05df91f6ea
Gerrit-Change-Number: 26596
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211216/67a8dba0/attachment.htm>


More information about the gerrit-log mailing list