[PATCH] libosmo-abis[master]: Resync RTP session in case of timestamp jump

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

Max gerrit-no-reply at lists.osmocom.org
Tue Oct 11 16:43:34 UTC 2016


Hello Harald Welte, Jenkins Builder,

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

    https://gerrit.osmocom.org/1028

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

Resync RTP session in case of timestamp jump

This will prevent the timestamp jump to occur between each call for DTX
if something goes wrong. It's handy as oRTP does not manage well
timestamp jump if it is not resynced.

Fixes: OS#1803
Change-Id: Iae5da832218dbd1ede7a9451102074f2a5eb66df
---
M src/trau/osmo_ortp.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/28/1028/3

diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c
index 5441337..68db5da 100644
--- a/src/trau/osmo_ortp.c
+++ b/src/trau/osmo_ortp.c
@@ -139,7 +139,8 @@
 	uint32_t ts = rtp_session_get_current_recv_ts(rs);
 
 	LOGP(DLMIB, LOGL_NOTICE,
-	     "osmo-ortp(%d): timestamp_jump, new TS %d\n", port, ts);
+	     "osmo-ortp(%d): timestamp_jump, new TS %d, resyncing\n", port, ts);
+	rtp_session_resync(rs);
 }
 
 

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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iae5da832218dbd1ede7a9451102074f2a5eb66df
Gerrit-PatchSet: 3
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list