Hi Daniel,
Yes. Improper length check.
Regards,
Seudin
On Mon, Jan 12, 2015 at 3:44 PM, Daniel-Constantin Mierla <miconda(a)gmail.com
wrote:
Hello,
that fix was actually changing:
if (c < '0' && c > '9') return -1;
to:
if (c < '0' || c > '9') return -1;
The rest was auto-indentation by the editor.
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:
for (i = 1; i <= _user->len; i++) {
Is it?
Cheers,
Daniel
On 12/01/15 15:34, Seudin Kasumovic wrote:
Hi,
The fix below is wrong, condition must be '<' instead of '<='.
https://github.com/kamailio/kamailio/commit/fbcfa198a2c89868fa021f6a6377442…
and backported in 4.2
https://github.com/kamailio/kamailio/commit/2d1032b3c8a1184357615b37da99dfd…
Regards,
--
Seudin Kasumovic
_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing
listsr-users@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
--
Daniel-Constantin
Mierlahttp://twitter.com/#!/miconda -
http://www.linkedin.com/in/miconda
_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
--
MSC Seudin Kasumovic
Tuzla, Bosnia