[PATCH] libosmo-abis[master]: Fix RTP jitter buffer that never stop to increase.

[jfdionne]

Hello Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/1982

to look at the new patch set (#2).

Fix RTP jitter buffer that never stop to increase.

Duration passed to osmo_rtp_send_frame_ext function is based
on the last frame and the current one. Duration must then be
added to the timestamp before being transmitted.

Change-Id: I0593d6530c097cca34125a0ae2dd1b019b4dd80d
---
M src/trau/osmo_ortp.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/82/1982/2

diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c
index d4d4ff5..0dd08e5 100644
--- a/src/trau/osmo_ortp.c
+++ b/src/trau/osmo_ortp.c
@@ -459,9 +459,9 @@
 		return -ENOMEM;
 
 	rtp_set_markbit(mblk, marker);
+	rs->tx_timestamp += duration;
 	rc = rtp_session_sendm_with_ts(rs->sess, mblk,
 				       rs->tx_timestamp);
-	rs->tx_timestamp += duration;
 	if (rc < 0) {
 		/* no need to free() the mblk, as rtp_session_rtp_send()
 		 * unconditionally free()s the mblk even in case of

-- 
To view, visit https://gerrit.osmocom.org/1982
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0593d6530c097cca34125a0ae2dd1b019b4dd80d
Gerrit-PatchSet: 2
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Owner: jfdionne <jf.dionne at nutaq.com>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: jfdionne <jf.dionne at nutaq.com>