> Can someone help review this fix? Changed the required-mechanism check by > checking if the particular mechanism is inside the list of enabled supported > mechanisms. This should be more reliable than calling C_GetMechanismInfo(..) > on the required mechanism given vendors may return various sorts of error > codes. > > Thanks, > Valerie
Valerie Peng has updated the pull request incrementally with one additional commit since the last revision: Address review comments and reverted back to iterating through all available mechanisms. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/20207/files - new: https://git.openjdk.org/jdk/pull/20207/files/a37ab014..8f3bdbbc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=20207&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20207&range=00-01 Stats: 27 lines in 3 files changed: 7 ins; 16 del; 4 mod Patch: https://git.openjdk.org/jdk/pull/20207.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/20207/head:pull/20207 PR: https://git.openjdk.org/jdk/pull/20207