This looks fine, just one comment. Can you put braces around the if debug statement on lines 117-118 (just to be safe).

--Sean

On 03/25/2015 09:52 AM, Seán Coffey wrote:
This issue was reported while debugging a pkcs11 issue. While I can't
reproduce it, the captured stack trace does show a deadlock issue. Fix
is to simply stop using the SessionManager class lock to record and
print the maxActiveSessions in use and replace it with a local lock.

bug: https://bugs.openjdk.java.net/browse/JDK-8059588
webrev : http://cr.openjdk.java.net/~coffeys/webrev.8059588/webrev/

Regards,
Sean.

Reply via email to