<p>atomic_cmpxchg returns the previous value.  If the previous value was 0 then it was previously unlocked and we now have acquired the lock.</p>

<p>Thus, it should  be c==0 that is considered as the case we enter the lock, not c!=0</p>

<p>As far as I can tell, lock_try (and lock_set_try) are currently no used anywhere in the code, so this fix shoudn't have any impact</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/kamailio/kamailio/pull/520'>https://github.com/kamailio/kamailio/pull/520</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>core/futex: fix bug in futex_try</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/520/files#diff-0">futexlock.h</a>
    (2)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/kamailio/kamailio/pull/520.patch'>https://github.com/kamailio/kamailio/pull/520.patch</a></li>
  <li><a href='https://github.com/kamailio/kamailio/pull/520.diff'>https://github.com/kamailio/kamailio/pull/520.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/kamailio/kamailio/pull/520">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZeB7Bk8aLrXofFFN03d1Gj6WcSHPks5pnrk5gaJpZM4HiiyO.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/pull/520"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>