Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aes/asm/aesni-sha*-x86_64.pl: fix IV handling in SHAEXT paths. #2992

Closed
wants to merge 1 commit into from

Conversation

dot-asm
Copy link
Contributor

@dot-asm dot-asm commented Mar 20, 2017

Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.

Should fix #2975.

Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.
@dot-asm dot-asm added branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch 1.1.0 branch: master Merge to master branch labels Mar 20, 2017
@levitte levitte added the approval: done This pull request has the required number of approvals label Mar 20, 2017
levitte pushed a commit that referenced this pull request Mar 22, 2017
Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2992)
levitte pushed a commit that referenced this pull request Mar 22, 2017
Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2992)

(cherry picked from commit 08d0962)
@dot-asm dot-asm closed this Mar 22, 2017
levitte pushed a commit that referenced this pull request Mar 22, 2017
Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2992)

(cherry picked from commit 08d0962)
pracj3am pushed a commit to cdn77/openssl that referenced this pull request Aug 22, 2017
Initial IV was disregarded on SHAEXT-capable processors. Amazingly
enough bulk AES128-SHA* talk-to-yourself tests were passing.

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from openssl#2992)

(cherry picked from commit 08d0962)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Intel SHA extensions (Goldmont PCUs) possibly broken
2 participants