<p><a href="https://gerrit.asterisk.org/10390">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10390/1/main/astobj2_container.c">File main/astobj2_container.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10390/1/main/astobj2_container.c@771">Patch Set #1, Line 771:</a> <code style="font-family:monospace,monospace">        if (!(flags & OBJ_NOLOCK)) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The test was not inverted. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Actually now that I'm looking at this again why do we need to lock clone at all?  The only pointer to clone is the local variable so it's impossible for any other thread to access clone.  Locking inside ao2_container_dup makes sense but in this function it seems pointless.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10390">change 10390</a>. To unsubscribe, or for help writing mail filters, 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/10390"/><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: I9662b82eb39e7627a1f1944fd18f967a2b987344 </div>
<div style="display:none"> Gerrit-Change-Number: 10390 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 03 Oct 2018 13:21:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>