<p>Hey <a href="https://github.com/bogdan-iancu" class="user-mention">@bogdan-iancu</a>, thanks for chiming in :-)</p>

<blockquote>
<p>that may simplify your patch a lot - the nat_helper timer routing is registered with "TIMER_FLAG_DELAY_ON_DELAY" - this guarantees that (even executed in various processes) the handler will not be run in parallel (all its executions will be serial). So you do not need locking at all. Also, the counter itself can shm allocated and its pointer can be passed to the timer handler (more elegant versus a global var).</p>
</blockquote>

<p>This sounds like a better approach to me, can you adjust to it <a href="https://github.com/aerringer" class="user-mention">@aerringer</a>? Thanks a lot!</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/OpenSIPS/opensips/pull/751#issuecomment-171261698">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AFOciRClbQDOlUJNpcZBInIX91WRVxhlks5pZi3RgaJpZM4HCenG.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/OpenSIPS/opensips/pull/751#issuecomment-171261698"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>