changeset 23033:b6ca1802dc7c

8079644: memory stomping error with ResourceManagement and TestAgentStress.java Summary: the cached class file structure must be deallocated instead of the cached class file bytes Reviewed-by: coleenp, sla
author sspitsyn
date Wed, 20 May 2015 02:57:25 -0700
parents 37a5a1341478
children 5efc25c36716 9904bb920313
files src/share/vm/prims/jvmtiRedefineClasses.cpp
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/prims/jvmtiRedefineClasses.cpp	Tue May 19 11:06:34 2015 +0200
+++ b/src/share/vm/prims/jvmtiRedefineClasses.cpp	Wed May 20 02:57:25 2015 -0700
@@ -3980,8 +3980,8 @@
            the_class->get_cached_class_file_bytes()) {
     // The same class can be present twice in the scratch classes list or there
     // are multiple concurrent RetransformClasses calls on different threads.
-    // In such cases we have to deallocate scratch_class cached_class_file_bytes.
-    os::free(scratch_class->get_cached_class_file_bytes());
+    // In such cases we have to deallocate scratch_class cached_class_file.
+    os::free(scratch_class->get_cached_class_file());
   }
 
   // NULL out in scratch class to not delete twice.  The class to be redefined