RFR: 8238274: (sctp) JDK-7118373 is not fixed for SctpChannel [v2]
Masanori Yano
myano at openjdk.java.net
Tue Sep 14 10:19:18 UTC 2021
On Mon, 13 Sep 2021 13:18:10 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Masanori Yano has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - 8238274: (sctp) JDK-7118373 is not fixed for SctpChannel
>> - Merge branch 'master' of https://github.com/masyano/jdk into 8238274
>> - 8238274: (sctp) JDK-7118373 is not fixed for SctpChannel
>
> test/jdk/com/sun/nio/sctp/SctpChannel/CloseDescriptors.java line 29:
>
>> 27: * @summary Potential leak file descriptor for SCTP
>> 28: * @requires (os.family == "linux")
>> 29: * @run main CloseDescriptors
>
> The test SctpMultiChannel/CloseDescriptors seems to work - at least it doesn't fail all the time - but it's using `@run main/othervm`. I believe you should make this test run in /othervm mode for more stability. I will test again with your later change and /othervm mode.
I pushed a fix to run the test in othervm mode. Please test again.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5274
More information about the net-dev
mailing list