[MERGED] 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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Wed Oct 12 09:26:01 UTC 2016


Harald Welte has submitted this change and it was merged.

Change subject: Resync RTP session in case of timestamp jump
......................................................................


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(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c
index 6ce70d2..e3c6234 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: merged
Gerrit-Change-Id: Iae5da832218dbd1ede7a9451102074f2a5eb66df
Gerrit-PatchSet: 4
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