RFR: JDK-8040810 - Uninitialised memory in jdk/src/windows/native/java/net: net_util_md.c, TwoStacksPlainSocketImpl.c, TwoStacksPlainDatagramSocketImpl.c, DualStackPlainSocketImpl.c, DualStackPlainDatagramSocketImpl.c

Alan Bateman Alan.Bateman at oracle.com
Wed Jul 16 17:28:23 UTC 2014


On 16/07/2014 18:00, Mark Sheppard wrote:
> Hi
>     please oblige and review the following changes
>
> http://cr.openjdk.java.net/~msheppar/8040810/webrev/
>
> which address the issue raised in
>
> https://bugs.openjdk.java.net/browse/JDK-8040810
>
> resulting from static code analysis.
>
> these changes explicitly initialize local function variables, which 
> are in the main
> out parameters to other function calls and hence are set within the 
> called function.
> It can be reasonably argued that the initialization is unnecessary, 
> but current coding
> guidance is to perform the initialization
>
I assume in NET_Bind that it isn't necessary to initialize rv, same 
thing in NET_SocketClose. Otherwise looks okay.

-Alan



More information about the net-dev mailing list