<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/9379">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Pau Espin Pedrol: Looks good to me, but someone else must approve
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ortp: reset rtp session on SSRC changes<br><br>When libortp detects a timestamp jump, we resynchronize the session<br>in order to restore proper rtp frame reception. However, we do not<br>resynchronize on an SSRC change. An SSRC change usually indicates<br>the change of an RTP stream source, which is a much more profound<br>event than a timestamp change, so we should even reset the session<br>on SSRC changes.<br><br>Also, not resynchronizing the session on an SSRC change causes<br>problems when the jitter buffer feature is used. In those cases<br>it takes libortp a long time until it finally detects the timestamp<br>jump and the session resyhcornization happens late.<br><br>- reset the ession on SSRC changes by calling rtp_session_reset()<br><br>Change-Id: I8688a55cd20d0e14493c357db80754851e12f2fb<br>Related OS#3299<br>---<br>M src/trau/osmo_ortp.c<br>1 file changed, 2 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c</span><br><span>index 18a6123..9062289 100644</span><br><span>--- a/src/trau/osmo_ortp.c</span><br><span>+++ b/src/trau/osmo_ortp.c</span><br><span>@@ -114,7 +114,8 @@</span><br><span>   uint32_t ssrc = rtp_session_get_recv_ssrc(rs);</span><br><span> </span><br><span>   LOGP(DLMIB, LOGL_INFO,</span><br><span style="color: hsl(0, 100%, 40%);">-       "osmo-ortp(%d): ssrc_changed to 0x%08x\n", port, ssrc);</span><br><span style="color: hsl(120, 100%, 40%);">+             "osmo-ortp(%d): ssrc_changed to 0x%08x, resetting\n", port, ssrc);</span><br><span style="color: hsl(120, 100%, 40%);">+     rtp_session_reset(rs);</span><br><span> }</span><br><span> </span><br><span> static void ortp_sig_cb_pt(RtpSession *rs, void *data)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9379">change 9379</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/9379"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-abis </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I8688a55cd20d0e14493c357db80754851e12f2fb </div>
<div style="display:none"> Gerrit-Change-Number: 9379 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>