RFR: 8270317: Large Allocation in CipherSuite [v3]

Clive Verghese cverghese at openjdk.java.net
Thu Jul 22 19:01:05 UTC 2021


On Thu, 22 Jul 2021 18:50:08 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> Clive Verghese has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add allowed and default lists
>
> src/java.base/share/classes/sun/security/ssl/CipherSuite.java line 866:
> 
>> 864:         List<CipherSuite> allowedCS = new ArrayList<>();
>> 865:         List<CipherSuite> defaultCS = new ArrayList<>();
>> 866:         for(CipherSuite cs : CipherSuite.values()) {
> 
> I may add blank lines before and after the for-loop block.

Than you @XueleiFan for approving the PR. 

I have added the black lines before and after the PR.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4783



More information about the security-dev mailing list