Looks good to me, thanks!

Roman

> Bug:
>   https://bugs.openjdk.java.net/browse/JDK-8241750
> 
> Fix:
> 
> diff -r fef47d126675 src/jdk.jdwp.agent/share/native/libjdwp/classTrack.c
> --- a/src/jdk.jdwp.agent/share/native/libjdwp/classTrack.c      Fri Mar 27 
> 15:33:24 2020 +0100
> +++ b/src/jdk.jdwp.agent/share/native/libjdwp/classTrack.c      Fri Mar 27 
> 17:47:31 2020 +0100
> @@ -70,5 +70,5 @@
>        return;
>      }
> -    *(char**)bagAdd(deletedSignatures) = (char*)tag;
> +    *(char**)bagAdd(deletedSignatures) = (char*)jlong_to_ptr(tag);
> 
>      debugMonitorExit(classTrackLock);
> @@ -118,5 +118,5 @@
>          EXIT_ERROR(error,"signature");
>      }
> -    error = JVMTI_FUNC_PTR(trackingEnv, SetTag)(env, klass, 
> (jlong)signature);
> +    error = JVMTI_FUNC_PTR(trackingEnv, SetTag)(env, klass, 
> ptr_to_jlong(signature));
>      if (error != JVMTI_ERROR_NONE) {
>          jvmtiDeallocate(signature);
> 
> Testing: Linux {x86_64, x86_32} x {builds, vmTestbase_nsk_jdwp}; jdk-submit 
> (running)
> 

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to