<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6642">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/6642/2//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/6642/2//COMMIT_MSG@15">Patch Set #2, Line 15:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Per feedback changed "privacy==NULL" to "!privacy". Will look into<br>how to cherry pick to 14/15/master (and chan_sip.c patch to 11 LTS).<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove these two lines.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The commit message is part of the review and is EXACTLY what will be committed.  You should not chronicle changes to your patch in the commit message.  That is what gerrit is doing for you automatically.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/6642/2/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/6642/2/channels/chan_sip.c@18029">Patch Set #2, Line 18029:</a> <code style="font-family:monospace,monospace">    if (!ast_strlen_zero(privacy) && strncmp(privacy, "none", 4)) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think the comparison should be:<br>if (!ast_strlen_zero(privacy) && strcasecmp(privacy, "none")) {<br>}</p><p style="white-space: pre-wrap; word-wrap: break-word;">The privacy[] string will always be null terminated and if it isn't "none" then the number should be prohibited.  We probably should be case insensitive just in case.  We certainly are in the pjsip code.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6642">change 6642</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/6642"/><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: If438a21f31a962da32d7a33ff33bdeb1e776fe56 </div>
<div style="display:none"> Gerrit-Change-Number: 6642 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dtryba <daniel@tryba.nl> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 03 Oct 2017 16:59:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>