RFR 8168405: Pending exceptions in java.base/windows/native

Langer, Christoph christoph.langer at sap.com
Thu Oct 20 13:03:07 UTC 2016


Hi Pavel,

overall this looks good. I've got a few minor remarks:

1. What about using the macro CHECK_NULL_RETURN in NetworkInterface_winXP.c?

2. in Java_java_net_TwoStacksPlainDatagramSocketImpl_peekData:

You could move
1178     /* make sure receive() picks up the right fd */
1179     (*env)->SetIntField(env, this, pdsi_fduseID, fduse);
into the else branch above and set port to -1 in the n<0 branches. That way -1 will be returned always and the free(fullPacket) is also invoked if necessary with less lines of code :) But that's probably a matter of taste ;-)

Best regards
Christoph

> -----Original Message-----
> From: Pavel Rappo [mailto:pavel.rappo at oracle.com]
> Sent: Donnerstag, 20. Oktober 2016 14:47
> To: OpenJDK Network Dev list <net-dev at openjdk.java.net>
> Subject: RFR 8168405: Pending exceptions in java.base/windows/native
> 
> Hello,
> 
> Could you please review the following change for [1]?
> 
>    http://cr.openjdk.java.net/~prappo/8168405/webrev/
> 
> This change addresses some code paths in the native networking code for
> Windows
> operating system where thrown exceptions might be left unnoticed.
> 
> Thanks,
> -Pavel
> 
> --------------------------------------------------------------------------------
> [1] https://bugs.openjdk.java.net/browse/JDK-8168405



More information about the net-dev mailing list