RFR: 8330842: Support AES CBC with Ciphertext Stealing (CTS) in SunPKCS11 [v2]
Francisco Ferrari Bihurriet
fferrari at openjdk.org
Fri May 24 11:50:04 UTC 2024
On Thu, 23 May 2024 20:20:51 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
>> @theRealAph: no, it looked to me easier to read if we have input-related parameters separated from output ones. So I did this reordering in a separate commit (0a777e94229723376e1264e87cbf0ba805dc736f), to distinguish it from the core change (in c4419bfecc0b795f6a1bdcbe5c5a6aa0142350ea).
>>
>> We can revert 0a777e94229723376e1264e87cbf0ba805dc736f if it confuses more than it improves.
>
> I prefer to not making changes like this.
Reverted, see 68197517ec2f3e7e27debf6d2fca15a4b43c4e4e (whitespace-only change).
NOTE: new calls of `C_EncryptUpdate`/`C_DecryptUpdate`/`C_EncryptFinal`/`C_DecryptFinal` were retained in the re-arrangement style, as I found that we didn't introduce this re-arrangement, it was already present in most of the calls inside `P11Cipher::implUpdate()` and `P11Cipher::implDoFinal()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18898#discussion_r1613349318
More information about the security-dev
mailing list