<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;">Actually now that I'm looking at this again why do we need to lock clone at all?  The only pointer t […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You are going to run into issues with rwlocks if it is not locked.  __adjust_lock() has to assume that the lock is held for rwlocks to know how to adjust the lock.  You have stated that the lock is already held by external code when you pass OBJ_NOLOCK.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It is not like there will be any contention for the lock in order to lock it.</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 15:44:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>