<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    spotted an interesting blog on the MSDN timeout issue here : <br>
<a class="moz-txt-link-freetext" href="https://www.frameflow.com/ping-utility-flaw-in-windows-api-creating-false-timeouts/">https://www.frameflow.com/ping-utility-flaw-in-windows-api-creating-false-timeouts/</a><br>
    <pre class="moz-signature" cols="72">Regards,
Sean.</pre>
    <div class="moz-cite-prefix">On 21/09/16 17:42, Mark Sheppard wrote:<br>
    </div>
    <blockquote
      cite="mid:5d75efeb-3c4a-5674-1c76-d71607e759e4@oracle.com"
      type="cite">
      <meta content="text/html; charset=windows-1252"
        http-equiv="Content-Type">
      <br>
      the IcmpSendEcho series of calls come with some idiosyncrasies in
      that there is a minimum timeout that they can handle<br>
      think it is about 1000msecs. isReachable can specify a finer
      grained timeout hence the need for timeout check<br>
      <br>
      regards<br>
      Mark<br>
      <br>
      <div class="moz-cite-prefix">On 21/09/2016 17:18, Vyom Tewari
        wrote:<br>
      </div>
      <blockquote
        cite="mid:0a27f57b-1dfe-2e81-dd1f-0e34692397cc@oracle.com"
        type="cite">
        <meta content="text/html; charset=windows-1252"
          http-equiv="Content-Type">
        <p>Hi Rob,</p>
        <p>Do you really think this extra check is required ?</p>
        <pre><span class="new">if (pEchoReply->Status == IP_SUCCESS</span>
<span class="new">+                && (int)pEchoReply->RoundTripTime <= timeout)

I did not found any doc(MSDN) which explains this. I think in case of IP_SUCCESS "RoundTripTime"  is always less than timeout. 

I think similar changes is required in Inet6Address.c as well ?

Thanks,
Vyom
</span></pre>
        <br>
        <div class="moz-cite-prefix">On Wednesday 21 September 2016
          08:46 PM, Rob McKenna wrote:<br>
        </div>
        <blockquote cite="mid:20160921151646.GD5168@vimes" type="cite">
          <pre wrap="">Hi folks,

I'd like to fix a bug caused by an incorrect assumption. The IcmpSendEcho* calls can actually return a similar set of errors regardless of whether the call itself failed or succeeded. This change checks that both the call and the ping were successful. In addition to that it takes a number of common failure causes into account before deciding to throw an exception.

<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Erobm/8159410/webrev.01/">http://cr.openjdk.java.net/~robm/8159410/webrev.01/</a>

        -Rob
</pre>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>