[PATCH] JDK-8226976 - SessionTimeOutTests uses == operator for String value check

Jaikiran Pai jai.forums2013 at gmail.com
Tue Jul 2 02:05:22 UTC 2019


Thank you Xuelei. Here's the updated webrev
http://cr.openjdk.java.net/~jpai/webrev/8226976/01/webrev/. Updated the
copyright year and also added you as the reviewer, in the commit log
message.

-Jaikiran

On 01/07/19 10:17 PM, Xuelei Fan wrote:
> Hi Jaikiran,
>
> Would you mind update the copyright date to 2019?  Otherwise, looks
> good to me.
>
> Thanks,
> Xuelei
>
> On 6/29/2019 6:03 AM, Jaikiran Pai wrote:
>> While investigating a potential regression in JDK 13+, I happened to use
>> the test/jdk/javax/net/ssl/SSLSession/SessionTimeOutTests.java. This
>> test incorrectly uses the == operator for one of the String checks. I
>> have filed a bug[1] with the details and have a patch with the fix
>> here[2].
>>
>> Can I please get a review and a sponsor for this change?
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8226976
>>
>> [2] http://cr.openjdk.java.net/~jpai/webrev/8226976/0/webrev/
>>
>> -Jaikiran
>>
>>



More information about the security-dev mailing list