RFR: JDK-8320168: handle setsocktopt return values [v3]
Alan Bateman
alanb at openjdk.org
Fri Nov 17 09:55:31 UTC 2023
On Fri, 17 Nov 2023 08:07:02 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
> Hi Alan, do you want an additional change to src/java.base/windows/native/libnio/ch/WindowsAsynchronousServerSocketChannelImpl.c ? Was not really sure about this after your comment about this chunk .
I think it's okay. The only concern now will be third party LSPs but it would be months or years before such issues come out of the woodwork, so I think what you have in the WindowsAsynchronous* files is okay, just check for SOCKET_ERROR instead of < 0.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16684#issuecomment-1816055621
More information about the nio-dev
mailing list