<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7430">View Change</a></p><p>Patch set 1:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7430/1/channels/chan_sip.c">File channels/chan_sip.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7430/1/channels/chan_sip.c@34132">Patch Set #1, Line 34132:</a> <code style="font-family:monospace,monospace">  if (((peer->transports & peer2->transports) &</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">So should TCP/TLS not use INSECURE_PORT?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">For TCP/TLS, the same connection must be re-used. I see no benefit in insecure=port for TCP/TLS.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, David wrote something like "[For TCP,] the received packet's destination port will not match the one the peer table is built with." No idea what he was about. It is a pity that we cannot ask David anymore.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The code before this change looks like insecure=port is enabled for TCP/TLS always, regardless the settings in sip.conf. However, that might have been not a TCP/TLS issue in general but a specific issue at that time within Asterisk.</p><p style="white-space: pre-wrap; word-wrap: break-word;">By the way, did you see my previous reply in master? When peer2 has, and peer1 does not have insecure=port, zero is returned. Instead, it should go through the port test and return zero only when the ports do not match. Or? With other words, only when UDP and both peers have insecure=port, then a match should be returned. Otherwise the ports should be checked.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7430">change 7430</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.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.asterisk.org/7430"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Id190428bf1d931f2dbfd4b293f53ff8f20d98efa </div>
<div style="display:none"> Gerrit-Change-Number: 7430 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Dec 2017 17:19:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>