RFR 8080911: sun/security/krb5/auto/UseCacheAndStoreKey.java timed out intermittently

Xuelei Fan xuelei.fan at oracle.com
Mon May 25 23:59:33 UTC 2015


synchronized on class looks a little bit unsafe to me.  As singleton is
a static variable, creating the instance during initialization looks safer.

-   private static Config singleton = null;
+   private static Config singleton = new Config();

Xuelei

On 5/25/2015 10:16 PM, Weijun Wang wrote:
> Hi All
> 
> Please review a code change at
> 
>   http://cr.openjdk.java.net/~weijun/8080911/webrev.00/
> 
> I've limit the synchronized block to Config creation only and therefore
> won't deadlock with EType's class initialization.
> 
> Noreg-hard. The EType call is at class initialization and only run once
> in a VM session, which is extremely difficult to catch.
> 
> Thanks
> Max




More information about the security-dev mailing list