Skip to content

Comments

Fix curve448 issues#5418

Closed
mattcaswell wants to merge 2 commits intoopenssl:masterfrom
mattcaswell:fix-curve448-issues
Closed

Fix curve448 issues#5418
mattcaswell wants to merge 2 commits intoopenssl:masterfrom
mattcaswell:fix-curve448-issues

Conversation

@mattcaswell
Copy link
Member

Ubsan found some issues in the Curve448 code. It also broke no-ec. This fixes both issues.

We can't add NULL data into a hash
[extended tests]
@mattcaswell
Copy link
Member Author

Pushed. Thanks.

levitte pushed a commit that referenced this pull request Feb 21, 2018
We can't add NULL data into a hash

Reviewed-by: Rich Salz <[email protected]>
(Merged from #5418)
levitte pushed a commit that referenced this pull request Feb 21, 2018
[extended tests]

Reviewed-by: Rich Salz <[email protected]>
(Merged from #5418)
mspncp added a commit to mspncp/openssl that referenced this pull request Aug 2, 2018
Fixes openssl#6800
Replaces openssl#5418

This commit reverts commit 7876dbf and moves the check for a
zero-length input down the callstack into sha3_update().
levitte pushed a commit that referenced this pull request Aug 3, 2018
Fixes #6800
Replaces #5418

This commit reverts commit 7876dbf and moves the check for a
zero-length input down the callstack into sha3_update().

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Andy Polyakov <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #6838)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants