RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Wed Mar 5 11:51:00 UTC 2014
- Previous message: RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions
- Next message: RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 05/03/2014 11:28, Mark Sheppard wrote:
Hi please oblige and review the following changes http://cr.openjdk.java.net/~msheppar/8035930/webrev/
to address the issues raised in https://bugs.openjdk.java.net/browse/JDK-8035930 this adds additional NULL check for malloc and JNI calls and check for pending exceptions as appropriate Just two comments on this:
At line 213 then I think it would be better to return with the OOME pending rather than suppressing it and throwing a FNFE. In practical terms then if we are really out of native memory then it won't get very far but it would still be confusing to have FNF thrown for this case.
What you think about dropping "pathToNTPath: " from the exception messages? It's not clear to me that it helps and it also means these new exceptions are inconsistent with the other usages.
-Alan.
- Previous message: RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions
- Next message: RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]