RFR (xs) 8059588: deadlock in java/io/PrintStream when verbose javax.net.debug flags are set

[Bradford Wetmore]

Sean,

Is this a java.security.debug or a javax.net.debug issue as mentioned in 
the synopsis?  It imports from sun.security.util.Debug, which is 
java.security.debug, not javax.net.debug.

In the bug, both variables were set.

...deleted...
-Djavax.net.debug=ssl,handshake,record,keygen,session,defaultctx,sslctx,ses
sioncache,keymanager,trustmanager -Dcom.oracle.security.ucrypto.debug=true
-Djava.security.debug=all -Djsse.S

Probably the synopsis should be changed.

Brad



On 3/27/2015 12:05 PM, Seán Coffey wrote:
> Thanks Sean. Yes - I'll add those braces before pushing.
>
> regards,
> Sean.
>
> On 27/03/2015 18:58, Sean Mullan wrote:
>> 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.
>