Skip to content

Comments

Fix for no-ec and no-ec2m#4134

Closed
snhenson wants to merge 3 commits intoopenssl:masterfrom
snhenson:no-ec2m-fix
Closed

Fix for no-ec and no-ec2m#4134
snhenson wants to merge 3 commits intoopenssl:masterfrom
snhenson:no-ec2m-fix

Conversation

@snhenson
Copy link
Contributor

Checklist
  • tests are added or updated

This adds fixes for various warnings and failure with no-ec and no-ec2m. Will need backporting to 1.1.0 too.

Fix warning and don't use binary field certificate for ECDH CMS
key only test.
@snhenson snhenson added 1.1.0 branch: master Applies to master branch labels Aug 10, 2017
levitte pushed a commit that referenced this pull request Aug 10, 2017
Reviewed-by: Rich Salz <[email protected]>
(Merged from #4134)
levitte pushed a commit that referenced this pull request Aug 10, 2017
Reviewed-by: Rich Salz <[email protected]>
(Merged from #4134)
levitte pushed a commit that referenced this pull request Aug 10, 2017
Fix warning and don't use binary field certificate for ECDH CMS
key only test.

Reviewed-by: Rich Salz <[email protected]>
(Merged from #4134)
levitte pushed a commit that referenced this pull request Aug 10, 2017
Reviewed-by: Rich Salz <[email protected]>
(Merged from #4134)

(cherry picked from commit 1aee92b)
levitte pushed a commit that referenced this pull request Aug 10, 2017
Fix warning and don't use binary field certificate for ECDH CMS
key only test.

Reviewed-by: Rich Salz <[email protected]>
(Merged from #4134)

(cherry picked from commit ed5c7ea)
@snhenson snhenson closed this Aug 10, 2017
pracj3am pushed a commit to cdn77/openssl that referenced this pull request Aug 22, 2017
Reviewed-by: Rich Salz <[email protected]>
(Merged from openssl#4134)

(cherry picked from commit 1aee92b)
pracj3am pushed a commit to cdn77/openssl that referenced this pull request Aug 22, 2017
Fix warning and don't use binary field certificate for ECDH CMS
key only test.

Reviewed-by: Rich Salz <[email protected]>
(Merged from openssl#4134)

(cherry picked from commit ed5c7ea)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants