RFR: 8334297: (so) java/nio/channels/SocketChannel/OpenLeak.java should not depend on SecurityManager [v3]

[Daniel Fuchs]

On Fri, 14 Jun 2024 16:34:11 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Minor cleanup
>
> Looks good, assuming you've tested it on all platforms to ensure it is stable.

@AlanBateman My test campaign revealed that trying to connect to a reserved port on  the loopback didn't work for windows 2016 machines. I suspect that on that system you need to wait for the whole connect timeout before getting the connection refused exception.

I have reworked the test to only test with unresolved addresses on windows 2016, and we can use use the same technic of skipping the ConnectException if we find other platforms where this does not work out.

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

PR Comment: https://git.openjdk.org/jdk/pull/19723#issuecomment-2174203743