RFR: JDK-8036603 - Check jdk/src/windows/native/java/lang/ProcessEnvironment_md.c for JNI pending exceptions

Mark Sheppard mark.sheppard at oracle.com
Wed Mar 12 17:47:56 UTC 2014


Hi Alan,
    so L61 should be

if ((bytes = (*env)->NewByteArray(env, i)) == NULL) {
     FreeEnvironmentStringsA(blockA);
     return NULL;
}

regards
Mark


On 12/03/2014 17:24, Alan Bateman wrote:
> On 12/03/2014 17:18, Mark Sheppard wrote:
>> Hi
>>    an updated webrev as per suggested changes
>> http://cr.openjdk.java.net/~msheppar/8036603/webrev.01/
> This looks good. One other thing in this code (pre-dates your patch 
> but your patch makes it a bit more obvious) is that if NewByteArray 
> fails then we don't call FreeEnvironmentStringsA to free the environment.
>
> -Alan




More information about the core-libs-dev mailing list