RFR: JDK-8317603: provide handleSocketErrorWithMessage in windows Net.c for extended exception messages [v2]

[Vyom Tewari]

On Fri, 6 Oct 2023 07:21:13 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> On Windows, we miss a handleSocketErrorWithMessage function that provides an additional message showing what went wrong in the Net.c coding.  On Unix we have this function. 
>> This leads sometimes to exceptions like
>> 
>> MSG RTE: javax.naming.CommunicationException: example.com:1234 [Root exception is java.net.ConnectException: Connection timed out: no further information]
>> 
>> see https://bugs.openjdk.org/browse/JDK-8317307
>> 
>> It would be better to have a message explaining the reason instead of "no further information" .
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   switch to NET_ThrowNew

src/java.base/windows/native/libnio/ch/Net.c line 589:

> 587:     n = getsockopt(fdval(env, fdo), IPPROTO_IP, IP_MULTICAST_IF, (void*)&in, &arglen);
> 588:     if (n == SOCKET_ERROR) {
> 589:         NET_ThrowNew(env, WSAGetLastError(), "setsockopt");

there is typo, third argument should be "getsockopt"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16057#discussion_r1348460558