On Fri, 27 Feb 2026 03:13:32 GMT, Koushik Muthukrishnan Thirupattur 
<[email protected]> wrote:

>> Artur Barashev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Bound the memory usage when decompressing CompressedCertificate
>
> src/java.base/share/classes/sun/security/ssl/CompressedCertificate.java line 
> 165:
> 
>> 163:             byte[] compressedCertMsg;
>> 164: 
>> 165:             // Don't use cache if certificate_request_context is 
>> present.
> 
> NIT: Can we add more to this comment that explains what certMsg[0] represents 
> and why it avoids caching?

Yes, I guess we can be a bit more verbose here, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28682#discussion_r2866030358

Reply via email to