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, &copy);
> 		  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