<p>pespin <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/libosmo-abis/+/24966">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  daniel: Looks good to me, but someone else must approve
  osmith: Looks good to me, but someone else must approve
  Hoernchen: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">osmo_ortp: Fix seqno reset to 0 upon ssrc_changed<br><br>It was spotted in an osmo-mgw instance receiving RTP from osmo-bts 1.2.3<br>that sometimes the seqno was reset to 0 (while tx_timestamp was kept<br>properly). This seems to be the only possible way osmo-bts can reset the<br>seqno to 0 (calling rtp_session_reset()), since osmo_rtp_socket_create<br>already sets it to other values and inside ortp rtp_session_reset() is<br>the only one setting it to 0.<br>If SSRC changes, we are fine resetting some Rx state, which is done by<br>calling rtp_session_resync. However, calling rtp_session_reset() is too<br>much, since it erases other stuff like Tx state, which sholdn't be<br>affected in this case, since we are still transmitting the same stream.<br><br>Related: OS#3104<br>Related: OS#3299<br>Related: SYS#5498<br>Fixes: 28eeb6bc9365bbe47125775fcb1f696317c22466<br><br>Change-Id: Ic6006b4991c85413b009b70f8f3a6f97c41990ec<br>---<br>M src/trau/osmo_ortp.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<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 b72f681..011f097 100644</span><br><span>--- a/src/trau/osmo_ortp.c</span><br><span>+++ b/src/trau/osmo_ortp.c</span><br><span>@@ -142,8 +142,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, resetting\n", port, ssrc);</span><br><span style="color: hsl(0, 100%, 40%);">-       rtp_session_reset(rs);</span><br><span style="color: hsl(120, 100%, 40%);">+             "osmo-ortp(%d): ssrc_changed to 0x%08x, resyncing\n", port, ssrc);</span><br><span style="color: hsl(120, 100%, 40%);">+     rtp_session_resync(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/c/libosmo-abis/+/24966">change 24966</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/c/libosmo-abis/+/24966"/><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-Change-Id: Ic6006b4991c85413b009b70f8f3a6f97c41990ec </div>
<div style="display:none"> Gerrit-Change-Number: 24966 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>