RFR: 8156715: TrustStoreManager does not buffer keystore input stream [v2]

Artur Barashev abarashev at openjdk.org
Mon Jun 23 19:00:51 UTC 2025


On Mon, 23 Jun 2025 18:43:12 GMT, Artur Barashev <abarashev at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java line 379:
>> 
>>> 377: 
>>> 378:             if (!"NONE".equals(descriptor.storeName)) {
>>> 379:                 try (BufferedInputStream fis =
>> 
>> Nit:
>> `ks` is for KeyStore
>> `fis` is for FileInputStream
>> so should `bis` be used for BufferedInputStream?
>
> Yes, I also noticed that, will update it together with a copyright, thanks!

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25920#discussion_r2162341419


More information about the security-dev mailing list