<p>pespin <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24862">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgw: rx CRCX: Avoid sending dummy rtp if remote address not provided<br><br>The following sequence of events was seen frequently in a osmo-mgw<br>instance running on the field with heavy traffic:<br>"""<br>endpoint:rtpbridge/1@mgw CRCX: creating new connection ...<br>mgcp_network.c:236 endpoint:rtpbridge/1@mgw CI:1C8CCFA9 Failed to send dummy RTP packet.<br>"""<br><br>Allegedly, that happens because at CRCX time the remote address may<br>still not be known, hence we end up trying to send a dummy rtp packet<br>to, for instance, host 0.0.0.0 port 0, which will of course fail.<br>Let's avoid sending it if the address is not yet known.<br><br>Similary, same issue could be seen during MDCX, since at MDCX we don't<br>necessarily need to have a valid addr+port (there could be several MDCX<br>and only last one set it).<br><br>Finally, the keepalive timer also needs the check, since it iterates<br>over all connections, and it could be that some is still not fully<br>configured.<br><br>Related: SYS#5498<br>Change-Id: I8ceafda691146823b12232b4a804b4ce74acbdc8<br>---<br>M include/osmocom/mgcp/mgcp_network.h<br>M src/libosmo-mgcp/mgcp_network.c<br>M src/libosmo-mgcp/mgcp_protocol.c<br>3 files changed, 16 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/62/24862/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24862">change 24862</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/osmo-mgw/+/24862"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8ceafda691146823b12232b4a804b4ce74acbdc8 </div>
<div style="display:none"> Gerrit-Change-Number: 24862 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>