pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-netif/+/30170 )
Change subject: osmux: Drop marker bit in forged RTP packets to fill gaps
......................................................................
osmux: Drop marker bit in forged RTP packets to fill gaps
If the last RTP we received from the user is a Marker bit (which is
expected to be the first in the batch), then there's no need to keep
marking the forged RTP packets to fill holes also as Marker bit.
Change-Id: I5cef6185afbfce748473a096e8ebabd9c9628e12
---
M src/osmux_input.c
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/src/osmux_input.c b/src/osmux_input.c
index 3a85b45..866588f 100644
--- a/src/osmux_input.c
+++ b/src/osmux_input.c
@@ -401,7 +401,8 @@
/* The original RTP message has been already sanity checked. */
rtph = osmo_rtp_get_hdr(clone);
-
+ /* Faking a follow up RTP pkt here, so no Marker bit: */
+ rtph->marker = false;
/* Adjust sequence number and timestamp */
rtph->sequence = htons(ntohs(rtph->sequence) + i);
rtph->timestamp = htonl(ntohl(rtph->timestamp) +
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-netif/+/30170
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I5cef6185afbfce748473a096e8ebabd9c9628e12
Gerrit-Change-Number: 30170
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged