RFR: 4887998: Use Integer.rotateLeft() and rotateRight() in crypto implementations
Weijun Wang
weijun at openjdk.org
Mon Jul 11 22:42:51 UTC 2022
On Mon, 11 Jul 2022 21:20:43 GMT, Mark Powers <duke at openjdk.org> wrote:
> https://bugs.openjdk.org/browse/JDK-4887998
Two comments:
1. There are many unnecessary parenthesis, like `(32 - s)` or `(Integer... + 5)`.
2. Why choose `rotateRight` and not `rotateLeft`? https://en.wikipedia.org/wiki/SHA-1 uses "circular left shift operation" to describe the rotation.
-------------
PR: https://git.openjdk.org/jdk/pull/9456
More information about the security-dev
mailing list