RFR: JDK-8035930 - Check jdk/src/windows/native/java/io/io_util_md.c for JNI pending exceptions

Mark Sheppard mark.sheppard at oracle.com
Wed Mar 5 19:47:54 UTC 2014


Hi
    webrev has been updated as per Alan's suggestions

http://cr.openjdk.java.net/~msheppar/8035930/webrev.01/

regards
Mark

On 05/03/2014 11:51, Alan Bateman wrote:
> 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:
>
> 1. 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.
>
> 2. 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.




More information about the core-libs-dev mailing list