Review request (S): 6539281 -Xcheck:jni should validate char*	argument to ReleaseStringUTFChars
    Staffan Larsen 
    staffan.larsen at oracle.com
       
    Mon Dec  6 00:32:52 PST 2010
    
    
  
Update webrev: http://cr.openjdk.java.net/~sla/6539281/webrev.01/
Changes in this version:
1) Use "const jint" instead of "#define".
2) GetStringChars/GetStringUTFChars: assert that 'isCopy' == JNI_TRUE
3) ReleaseStringChars/ReleaseStringUTFChars: check 'chars' for NULL
4) GetStringUTFChars : To reduce copying I added a method java_lang_String::as_utf8_string which takes a pre-allocated buffer. Thus, the number of copies for unchecked JNI is reduced to one (previously two) and two for checked JNI (previously three). This could potentially speed up unchecked JNI if GetStringUTFChars is heavily used.
Thanks,
/Staffan
    
    
More information about the hotspot-runtime-dev
mailing list