<div dir="ltr"><span style="color:rgb(80,0,80);font-size:13px">strtoul == "string to unsigned long" only accepts positive numbers</span><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 12, 2015 at 11:00 AM, Oleksiy Vyalov <span dir="ltr"><<a href="mailto:ovyalov@google.com" target="_blank">ovyalov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">REPOSITORY<br>
  rL LLVM<br>
<br>
================<br>
Comment at: /Users/vharron/ll/svn/lldb/source/Utility/UriParser.cpp:42<br>
@@ -42,1 +41,3 @@<br>
<span class="">+    char* end = nullptr;<br>
     int port_tmp = strtoul(port_buf, &end, 10);<br>
</span><span class="">+    if (*end != 0 || port_tmp > 65535)<br>
</span>----------------<br>
You may define port_tmp as auto instead of integer - otherwise if result of strtoul is greater than MAX_INT but less than max of unsigned long int it might be just a negative number.<br>
<br>
================<br>
Comment at: /Users/vharron/ll/svn/lldb/source/Utility/UriParser.cpp:43<br>
@@ -42,3 +42,3 @@<br>
<span class="">     int port_tmp = strtoul(port_buf, &end, 10);<br>
-    if (*end != 0)<br>
+    if (*end != 0 || port_tmp > 65535)<br>
     {<br>
</span>----------------<br>
Check for portr_tmp <= 0?<br>
<br>
<a href="http://reviews.llvm.org/D6918" target="_blank">http://reviews.llvm.org/D6918</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><br><table cellspacing="0" cellpadding="0" style="font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Vince Harron |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Technical Lead Manager |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:vharron@google.com" target="_blank">vharron@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 858-442-0868</td></tr></tbody></table><br></div></div>
</div>