RFR: 8282077: PKCS11 provider C_sign() impl should handle CKR_BUFFER_TOO_SMALL error [v2]

Valerie Peng valeriep at openjdk.java.net
Sat Feb 19 00:07:34 UTC 2022


> Could someone please help review this trivial change? This is to add an error handling for the potential CKR_BUFFER_TOO_SMALL error when calling C_Sign(). Since none of the supported signature algorithms trigger this error as the default buffer size is large enough, this is more for consistency sake. No new regression test for this and thus the @noreg-hard label.
> 
> Thanks,
> Valerie

Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:

  free allocated 'ckpData' upon OOM.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7540/files
  - new: https://git.openjdk.java.net/jdk/pull/7540/files/19d1451a..67d70e9d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7540&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7540&range=00-01

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7540.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7540/head:pull/7540

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



More information about the security-dev mailing list