Review request (S): 8005002: Crash because of a raw oop in ClassLoaderData::add_dependency

[David Holmes]

Hi Stefan,

On 13/12/2012 6:16 PM, Stefan Karlsson wrote:
> http://cr.openjdk.java.net/~stefank/8005002/webrev.00/
>
> This is a fix for a crash seen the nightly comp testing in the test:
> vm/mlvm/indy/stress/gc/lotsOfCallSites
>
> We have a raw oop while calling out to an allocation function that might
> block for GC. If a GC enters, it might move the Object but will not
> update the oop. Make sure that we wrap the oop in a handle before
> calling the allocation function.

Looks good to me.

I guess CHECK_UNHANDLED_OOPS never got this one :(

Thanks,
David

> StefanK