<div dir="ltr">Hi Daniel,<div><br></div><div>Yes. Improper length check.</div><div><br></div><div>Regards,</div><div>Seudin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 12, 2015 at 3:44 PM, Daniel-Constantin Mierla <span dir="ltr"><<a href="mailto:miconda@gmail.com" target="_blank">miconda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    Hello,<br>
    <br>
    that fix was actually changing:<br>
    <br>
    if (c < '0' && c > '9') return -1;<br>
    <br>
    to:<br>
    <br>
    if (c < '0' || c > '9') return -1;<br>
    <br>
    The rest was auto-indentation by the editor.<br>
    <br>
    Providing the details of like or pasting the code where you refer
    would make it easier to understand what do you mean, but I guess you
    are referring to the line:<br>
    <br>
    for (i = 1; i <= _user->len; i++) {<br>
    <br>
    Is it?<br>
    <br>
    Cheers,<br>
    Daniel<div><div class="h5"><br>
    <br>
    <div>On 12/01/15 15:34, Seudin Kasumovic
      wrote:<br>
    </div>
    </div></div><blockquote type="cite"><div><div class="h5">
      <div dir="ltr">
        <div><br>
        </div>
        <div>Hi,</div>
        <div><br>
        </div>
        <div>The fix below is wrong, condition must be '<' instead of
          '<='.</div>
        <div><br>
        </div>
        <a href="https://github.com/kamailio/kamailio/commit/fbcfa198a2c89868fa021f6a63774425e480b27b" target="_blank">https://github.com/kamailio/kamailio/commit/fbcfa198a2c89868fa021f6a63774425e480b27b</a>
        <div>and  backported in 4.2</div>
        <div><a href="https://github.com/kamailio/kamailio/commit/2d1032b3c8a1184357615b37da99dfd76526afa7#diff-d7b03e3862e552e90fe6f958ec0f3a12" target="_blank">https://github.com/kamailio/kamailio/commit/2d1032b3c8a1184357615b37da99dfd76526afa7#diff-d7b03e3862e552e90fe6f958ec0f3a12</a><br>
        </div>
        <div><br>
        </div>
        <div><br>
        </div>
        <div>
          <div>Regards,</div>
          -- <br>
          <div>Seudin Kasumovic<br>
            <br>
          </div>
        </div>
      </div>
      <br>
      <fieldset></fieldset>
      <br>
      </div></div><pre>_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
<a href="mailto:sr-users@lists.sip-router.org" target="_blank">sr-users@lists.sip-router.org</a>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users</a><span class="HOEnZb"><font color="#888888">
</font></span></pre><span class="HOEnZb"><font color="#888888">
    </font></span></blockquote><span class="HOEnZb"><font color="#888888">
    <br>
    <pre cols="72">-- 
Daniel-Constantin Mierla
<a href="http://twitter.com/#!/miconda" target="_blank">http://twitter.com/#!/miconda</a> - <a href="http://www.linkedin.com/in/miconda" target="_blank">http://www.linkedin.com/in/miconda</a></pre>
  </font></span></div>

<br>_______________________________________________<br>
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list<br>
<a href="mailto:sr-users@lists.sip-router.org">sr-users@lists.sip-router.org</a><br>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">MSC Seudin Kasumovic<br>Tuzla, Bosnia</div>
</div>