RFR <8064470 > JNI exception pending in jdk/src/java/base/unix/native/libjava/FileDescriptor_md.c

Chris Hegarty chris.hegarty at oracle.com
Tue Sep 1 09:12:53 UTC 2015


Hi Vyom,

On 1 Sep 2015, at 09:38, Vyom Tewari <vyom.tewari at oracle.com> wrote:

> Hi everyone,
> 
> Can you please review my changes for below bug.
> 
> Bug:
>    JDK-8064470 : JNI exception pending in jdk/src/java/base/unix/native/libjava/FileDescriptor_md.c
> 
> Webrev:
> http://cr.openjdk.java.net/~dfuchs/vyom/8064470/webrev.01/

The changes look ok to me, and consistent with the Windows variant. Note that the final CHECK_NULL is not strictly necessary, but we seem to have adopted this style to avoid potential future mistakes if additional code is added.

-Chris.


> This change ensure that we immediately return if an exception has been raised (result == NULL) by GetMethodId() or proceed to the next
> line if there is no JNI pending exception.
> 
> if result is NULL, there is a pending exception, and we return. if result is not NULL, there is no pending exception, and we proceed to the next line...
> 
> Thanks,
> Vyom




More information about the core-libs-dev mailing list