RFR 8168405: Pending exceptions in java.base/windows/native
Pavel Rappo
pavel.rappo at oracle.com
Thu Oct 20 13:11:28 UTC 2016
> On 20 Oct 2016, at 14:03, Langer, Christoph <christoph.langer at sap.com> wrote:
>
> 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
Hi Christoph,
Thanks a lot for looking into this! I would prefer to be a bit more explicit in
both cases. And indeed I considered to do exactly what you described in the 2nd
point, but then I saw this pattern in several different places (just grep it) in
this file:
if (packetBufferLen > MAX_BUFFER_LEN) {
free(fullPacket);
}
So for consistency's sake maybe we should keep it as it is? I appreciate it
looks a bit wordy.
Thanks!
More information about the net-dev
mailing list