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

[Chris Hegarty]

On 27 Sep 2016, at 01:02, Mark Sheppard <mark.sheppard at oracle.com> wrote:
> 
> Hi Rob,
>    changes look reasonable …

+1

> perhaps align the two additions below the existing ERROR_XXX set, all neat and tidy :-)

+1

-Chris.

> 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
>