[8u] 8178870: instrumentation.retransformClasses cause coredump

[Zhengyu Gu]

Hi Aleksey,

On 8/20/19 11:57 AM, Aleksey Shipilev wrote:
> On 8/20/19 5:44 PM, Zhengyu Gu wrote:
>> On 8/20/19 10:23 AM, Aleksey Shipilev wrote:
>>> On 8/19/19 9:14 PM, Zhengyu Gu wrote:
>>>> 8u webrev: http://cr.openjdk.java.net/~zgu/JDK-8178870-8u/webrev.00/
>>>
>>> Backport looks good.
> 
> I just noticed your other backport has the new definition of get_ik here:
> 
> http://cr.openjdk.java.net/~zgu/JDK-8155951-8u/webrev.00/src/share/vm/prims/jvmtiRedefineClasses.cpp.sdiff.html
> 
> Maybe you want to commit them in more convenient order to get the definition in for this backport.

Nice catch! Updated webrev to reflect the order and eliminate the extra 
get_ik().

Webrev: http://cr.openjdk.java.net/~zgu/JDK-8178870-8u/webrev.01/

Thanks,

-Zhengyu



>