RFR: 8073108: GHASH Intrinsics
Vladimir Kozlov
vladimir.kozlov at oracle.com
Fri Feb 20 01:36:58 UTC 2015
JDK changes review.
-------------------
All exceptions should print invalid value as you did for state and
subkeyH checks. It would greatly help debugging.
Please, split first check into 3 separate checks and print invalid value
to know which condition failed.
Missing space after 'length': "invalid length"+subkeyH.length
Hotspot review.
---------------
Everything seems fine. Thank you for fixing the test.
What do you mean "Changed GHASHIntrinsics"? Don't pass intrinsic_id or
something else?
Thanks,
Vladimir
On 2/19/15 4:52 PM, Anthony Scarpino wrote:
> I've updated the webrevs. I believe I've added everyone's comments.
> - Changed GHASHIntrinsics and disabled for non-supported platforms
> - removing WIN64 ifdef in 32bit x86
> - fixed hotspot test so it can use compareArray()
> - modified the range checking
> - fixed processBlocks comment
>
> http://cr.openjdk.java.net/~ascarpino/8073108/hotspot/webrev.01/
>
> http://cr.openjdk.java.net/~ascarpino/8073108/jdk/webrev.01/
>
> thanks
>
> Tony
>
More information about the security-dev
mailing list