RFR: 8174729: Race Condition in java.lang.reflect.WeakCache

[Mandy Chung]

On Feb 23, 2017, at 2:28 AM, Andrew Dinn <adinn at redhat.com> wrote:
> 
> The following one line patch to the latest jdk8u tree fixes a race
> condition in java.lang.reflect.WeakCache:
> 
>  webrev:  http://cr.openjdk.java.net/~adinn/8174729/webrev.00
>  bug:     https://bugs.openjdk.java.net/browse/JDK-8174729
> 

This fix looks good to me.  Copyright header is missing in the new test.


> The webrev also includes a regression test, provided by Peter Levart the
> original author of WeakCache. It  fails consistently, albeit with no
> absolute guarantee, before the patch and never fails after patching.
> 
> Could someone sponsor this for inclusion in 8u and others review it
> (n.b. I am a JDK9 committer but not a JDK8 committer).

I can sponsor it.

Mandy