RFR: 8341346: Add support for exporting TLS Keying Material [v15]

Artur Barashev abarashev at openjdk.org
Thu May 15 12:53:54 UTC 2025


On Thu, 15 May 2025 00:27:31 GMT, Bradford Wetmore <wetmore at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/SSLSessionImpl.java line 1727:
>> 
>>> 1725: 
>>> 1726:             // context length must fit in 2 unsigned bytes.
>>> 1727:             if ((context != null) && context.length >= 65536) {
>> 
>> Nit: `context.length > 0xFFFF` would be more readable IMO
>
> Sure.

Also, the parentheses are not needed around `context != null`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24976#discussion_r2091102162


More information about the net-dev mailing list