<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7239">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>(3 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7239/2/main/acl.c">File main/acl.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7239/2/main/acl.c@481">Patch Set #2, Line 481:</a> <code style="font-family:monospace,monospace">             /* If this fails we set error, is that good enough? */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this is good enough.  An allocation error has already generated an ERROR message from the asterisk allocation functions.  PJSIP modules will destroy the config object on error because of the sorcery and config frameworks.  Chan_sip adds an additional log message complaining and keeps on plugging away.  Chan_iax2 and manager don't care about the error and keep plugging away.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7239/2/main/acl.c@570">Patch Set #2, Line 570:</a> <code style="font-family:monospace,monospace">        ast_debug(3, "%s/%s sense %u appended to ACL\n",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Unconditionally log this since it is already protected by DEBUG_ATLEAST(3).</p><p style="white-space: pre-wrap; word-wrap: break-word;">ast_log(LOG_DEBUG, "...");</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7239/2/main/acl.c@707">Patch Set #2, Line 707:</a> <code style="font-family:monospace,monospace">              ast_str_append(buf, 0, "/%s", ast_sockaddr_stringify_addr(&ha->netmask));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Just "%s" since the '/' is in the other ast_str_append().</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7239">change 7239</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/7239"/><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: Ia19eaaeb0b139ff7ce7b971c7550e85c8b78ab76 </div>
<div style="display:none"> Gerrit-Change-Number: 7239 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </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-Comment-Date: Fri, 17 Nov 2017 03:31:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>