RFR: 8275319 java.net.NetworkInterface throws java.lang.Error instead of SocketException [v4]

Daniel Jeliński duke at openjdk.java.net
Wed Oct 20 13:41:06 UTC 2021


On Wed, 20 Oct 2021 12:12:21 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adapt line length to 80 column display
>
> Hmm... JDK-8225239 might have introduced a bug. 
> I see that with the current code neither `enumAddresses_win` nor `enumAddresses_win_ipaddrtable` will return -2 - so there appears to be some dead code. I suspect that this line here is wrong:
>  https://github.com/openjdk/jdk/blob/a6c89e231872e7fab236ed87b5ccf44a189d6786/src/java.base/windows/native/libnet/NetworkInterface.c#L516
>  it should be returning `ret` not `NULL`

Thanks @dfuch and @AlanBateman for the review and all your help.

@dfuch I'd rather follow up on the remaining issues in a separate PR. This one is already doing more than the PR title advertises.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5956


More information about the net-dev mailing list