RFR: 8234466: Class loading deadlock involving X509Factory#commitEvent()

[Seán Coffey]

Thanks for the reviews. All callers of EventHelper log methods are 
ensuring that isLoggingSecurity() is true before proceeding. I've added 
an assert null check in the 4 logger methods to ensure expectations are 
in place.

http://cr.openjdk.java.net/~coffeys/webrev.8234466.v5/webrev/

Hope this helps,
Sean.

On 13/01/2020 14:38, Daniel Fuchs wrote:
> On 13/01/2020 14:06, Chris Hegarty wrote:
>> I’m going to ask, since I cannot find the answer myself. Why are some 
>> securityLogger::log invocations guarded with isLoggingSecurity, and 
>> others not? With this change shouldn’t all invocations be guarded, 
>> since it is isLoggingSecurity that assigns securityLogger a value?
>
> Argh! Well spotted chris!
>
> -- daniel