Code Review 6972374: NetworkInterface.getNetworkInterfaces throws "java.net.SocketException" on Solaris zone

Alan Bateman Alan.Bateman at oracle.com
Wed Jul 28 02:25:39 PDT 2010


Chris Hegarty wrote:
> Dmitry, Alan,
>
> The Solaris version of getFlags sets an Exception if the ioctl fails. 
> When used in addif getFlags will fail when access to the virtual 
> interface's parent is forbidden, i.e. in a zone. addif is called when 
> iterating over interfaces in enumIPvXInterfaces, if an exception 
> occurs it simply cleans up and returns, propagating the exception.
>
> getFalgs should not set an exception. All other calls to it check the 
> return value and set an exception if appropriate.
>
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6972374/webrev.00/webrev/
>
> Thanks,
> -Chris.
Looks okay to me. I assume the JNIEnv parameter is no longer needed.

It seems to me that we've had a slew of issues with NetworkInterface 
lately. Would it be worth re-reviewing the changes for 6931566 in case 
there are more? Ideally we should improve the test coverage to catch 
more issues but NetworkInterface is tricky to test completely due to 
variety of configurations in the wild.

-Alan.



More information about the net-dev mailing list