You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Leo Li (JIRA)" <ji...@apache.org> on 2007/01/16 09:24:27 UTC
[jira] Updated: (HARMONY-3008) [classlib][instrument]Releases byte
array too early in the native code of redefine classes.
[ https://issues.apache.org/jira/browse/HARMONY-3008?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Leo Li updated HARMONY-3008:
----------------------------
Attachment: patch-3008.diff
> [classlib][instrument]Releases byte array too early in the native code of redefine classes.
> -------------------------------------------------------------------------------------------
>
> Key: HARMONY-3008
> URL: https://issues.apache.org/jira/browse/HARMONY-3008
> Project: Harmony
> Issue Type: Bug
> Reporter: Leo Li
> Attachments: patch-3008.diff
>
>
> In instrument.c, Java_org_apache_harmony_instrument_internal_InstrumentationImpl_redefineClasses_1native, we have:
> Line 237:
> class_bytes = (*env)->GetByteArrayElements(env, jclass_bytes, ©);
> if(NULL == class_bytes){
> hymem_free_memory(class_definitions);
> return;
> }
> class_byte_count = (*env)->GetArrayLength(env, jclass_bytes);
> if(copy == JNI_TRUE){
> (*env)->ReleaseByteArrayElements(env,jclass_bytes, class_bytes, JNI_ABORT);
> }
>
> //construct a jvmtiClassDefinition element
> class_definitions[index].klass=klass;
> class_definitions[index].class_bytes=class_bytes;
> class_definitions[index].class_byte_count=class_byte_count;
> }
> //perform redefinition
> err=(*jvmti)->RedefineClasses(jvmti, length, class_definitions);
> the class_bytes are freed before RedefineClasses of JVM_TI uses them.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira