RFR: 8269481: SctpMultiChannel never releases own file descriptor [v2]

Brian Burkhalter bpb at openjdk.java.net
Tue Jun 29 17:08:05 UTC 2021


On Tue, 29 Jun 2021 16:55:33 GMT, Chris Hegarty <chegar at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8269481: Set state to KILLED *before* closing socket
>
> src/jdk.sctp/unix/classes/sun/nio/ch/sctp/SctpMultiChannelImpl.java line 377:
> 
>> 375:             if (state == ChannelState.UNINITIALIZED) {
>> 376:                 SctpNet.close(fdVal);
>> 377:                 state = ChannelState.KILLED;
> 
> Good catch. It seems that there was a missing state here, but not worth adding at this point.

I also thought that it looks like there is a missing state.

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

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


More information about the net-dev mailing list