6927816 : Demo crash in heaptracker with Non-Sun JDK due to possible violation of JNI spec

[Alan Bateman]

Steve Poole wrote:
> Hi all - I attach a hg export file with a fix for 6927816 for JDK7.   Since
> this is my first submission please be gentle.
>
> There is no testcase for this problem as it manifests only when using our
> J9 JVM.   I've build and run the change on Linux and Solaris using B121 and
> there doesn't seem to be any side effects.
>
> I'd appreciate a review of the change to make sure I've got it right - also
> some guidance on what to expect in seeing this fix progress through the
> process to the main jdk7 repository.
>
>
>
> (See attached file: 6927816.export)
>
>
> Regards
>
> Steve Poole
Looks good to me too.

In case you haven't seen them, the test cases for the jvmti demos are in 
jdk/test/demo/jvmti/ directory. You can use jtreg to run them.

I would suggest pushing it the TL repo [1] once your ssh key is setup 
and you have a username. Once your changes are in the TL repo then they 
will find their way into the master. It can take a few weeks, depending 
on when you push. At this time, the changes accumulated in TL are pushed 
to master every two weeks.

-Alan.

[1] http://hg.openjdk.java.net/jdk7/tl/jdk