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