looks fine to me. Thanks, Xuelei
> On Jul 7, 2016, at 3:33 PM, Sibabrata Sahoo <sibabrata.sa...@oracle.com> > wrote: > > Hi Xuelei, > > As suggested I have created a new bug to enable debug log for the failure > raised for JDK-8160210. Now the original bug JDK-8160210 will be under > observation until the failure found again in future. > > Because the fix is for a new bug, so the original subject line changed for > this email thread. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8160940 > Webrev: http://cr.openjdk.java.net/~ssahoo/8160940/webrev.00/ > > Change over previous: > This change set has no change over previous except adding "record" to JVM > param "-Djavax.net.debug ". > > Thanks, > Siba > > -----Original Message----- > From: Xuelei Fan > Sent: Tuesday, July 05, 2016 6:03 PM > To: Sibabrata Sahoo; security-dev@openjdk.java.net > Subject: Re: [9] RFR: 8160210: javax/net/ssl/HttpsURLConnection/Equals.java > failed with "SSL peer shut down incorrectly" > > You may also want to dump the SSL record so that we can analysis the record > directly. > > -Djavax.net.debug=ssl,handshake,record > > This update does not actually address the potential problem. Please file a > new bug for this update, and keep this bug open so that the issue can be > tracked. > > Thanks, > Xuelei > >> On 7/5/2016 7:20 PM, Sibabrata Sahoo wrote: >> Hi, >> >> >> >> Please review the patch for "8160210: >> javax/net/ssl/HttpsURLConnection/Equals.java failed with "SSL peer >> shut down incorrectly"" >> >> >> >> JBS: https://bugs.openjdk.java.net/browse/JDK-8160210 >> >> Webrev: http://cr.openjdk.java.net/~ssahoo/8160210/webrev.00/ >> >> >> >> Description: >> >> This is a rare failure occurred in Mac OS X 10.10.5 (x86_64) and has >> not been seen as frequent failure. From the test point of view it does >> not look anything wrong. >> >> It is a very rare failure and the failure log does not leave much clue >> about the failure. I am just adding few debug statements such as >> "-Djavax.net.debug=ssl,handshake", so that if it will fail in future >> it will have some meaningful information to debug. >> >> NOTE: The code has been formatted with NetBeans IDE and that is the >> reason it appears to have change almost in all line numbers. Please >> suggest if any possible fix can be made for this bug. >> >> >> >> Thanks, >> >> Siba >