<p>The issue solved by a previous patch was that is_e164() was returning always true -- it was checking that (c<'0' && c>'9') to say it is not e164, which never happens.</p>

<p>The issue solved by the patch on this item was about going over the length of the parameter with 1, which was even before the first patch.</p>

<p>After the first patch, the effect of second issue going over the length, results in checking an extra character which very likely is not a digit.</p>

<p>In other words, the first fix revealed another bug by enabling a side effect of it in most of the cases.</p>

<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/28#issuecomment-70110158">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36Zev8gfGdfWk3XNG50zjDhgeBqnmZks5nh944gaJpZM4DRNlf.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/28#issuecomment-70110158"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>