This looks good to me Mark.

Trivially, and it is more of a stylistic preference, towards the end of the 
Solaris NetworkInterface.c I would remove the if (ob) … else, and use 
CHECK_NULL_RETURN(obj, NULL). But, what you have is right, either is fine.

-Chris.

On 4 Mar 2014, at 11:06, Mark Sheppard <mark.shepp...@oracle.com> wrote:

> Hi
>  please oblige and review the following changes
> 
> http://cr.openjdk.java.net/~msheppar/8025293/webrev/
> 
> to address the issue in
> 
> https://bugs.openjdk.java.net/browse/JDK-8025293
> 
> this applies additional checks after JNI native calls in
> src/solaris/native/java/net/NetworkInterface.c
> src/windows/native/java/net/NetworkInterface.c
> 
> The main changes are NULL checks and pending exception checks.
> 
> regards
> Mark

Reply via email to