<div dir="ltr">my patch on lib/pending.c is only modifying  get_int_value() function.<div><div>Checking '-' and leading white spaces in the string.</div><div><br></div><div>I changed this</div><div><div>-  if (errno || numstr == ptr || *ptr || rvalue < lowrange || rvalue > highrange)</div>
<div>-    perror_exit("bad number '%s'", numstr);</div></div><div><br></div><div>to</div><div><br></div><div><div>+  if(errno || *ptr) perror_exit("invalid number '%s'", numstr);</div><div>
+  if(rvalue < lowrange || rvalue > highrange)</div><div>+    error_exit("out of range '%s'", numstr);</div></div><div><br></div><div>for having a better error message.</div><div><br></div><div>IPv6 is in fact important. Working on it, but was confused whether to have the IPv6 support</div>
<div>in the same traceroute.c or separate. As far as I see, there will be lot many if/else checks for IPv4/IPv6 cases.</div><div>Like for the incoming packet parsing.</div><div><br></div><div>Any suggestions are welcome.</div>
<div><br></div><div>regards,</div><div>Ashwini</div><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 13, 2013 at 11:31 PM, Isaac <span dir="ltr"><<a href="mailto:ibid.ag@gmail.com" target="_blank">ibid.ag@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 class="im">On Fri, Sep 13, 2013 at 03:10:24PM +0900, Ashwini Sharma wrote:<br>
> Hi All,<br>
><br>
> Attached is the traceroute patch.<br>
><br>
> This supports IPv4 destination trace.<br>
><br>
> Have a look at the same and pass on your comments.<br>
<br>
</div>I note that it appears to revert one of the recent changes to lib/pending.c.<br>
<br>
(Would you mind sending new toys as just the *.c file that goes in toys/*/<br>
when it doesn't need to touch lib/?)<br>
<br>
Also, isn't IPv6 rather important these days?<br>
<br>
Thanks,<br>
Isaac Dunham<br>
</blockquote></div><br></div>