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