pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve laforge: Looks good to me, approved
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(-)

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 change 30170. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I5cef6185afbfce748473a096e8ebabd9c9628e12
Gerrit-Change-Number: 30170
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged