RFR - 8166747: Add invalid network / computer name cases to isReachable known failure switch

Mark Sheppard mark.sheppard at oracle.com
Tue Sep 27 00:02:51 UTC 2016


Hi Rob,
     changes look reasonable ...
perhaps align the two additions below the existing ERROR_XXX set, all 
neat and tidy :-)

regards
Mark

On 27/09/2016 00:09, Rob McKenna wrote:
> Hi folks,
>
> Looking for a review of this simple addition to Inet4AddressImpl.c on Windows. As per the bug report:
>
> In the ping4() call in Inet4AddressImpl.c on Windows there is a switch statement containing failure codes for IcmpSendEcho which correspond to well known and expected failures for this call when a host is not reachable. In these cases ping4() simply returns false as opposed to throwing an exception.
>
> Prior releases of the JDK would return false when using the tcp ping method where we currently throw an exception with the ERROR_INVALID_COMPUTERNAME (Windows error code 1210) or ERROR_INVALID_NETNAME (1214) errors. We should add these cases to the switch statement for compatibility purposes.
>
> http://cr.openjdk.java.net/~robm/8166747/webrev.01/
>
> 	-Rob



More information about the net-dev mailing list