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

DOC: Modify one example in EVP_PKEY_fromdata(3) #12389

Closed
wants to merge 5 commits into from

Conversation

levitte
Copy link
Member

@levitte levitte commented Jul 7, 2020

The example to create an EC key from user data didn't show what one
could expect and application to do, especially with regard to how it's
done with raw EC functions. We therefore refactor it to make proper
use of a BIGNUM where expected, and also use OSSL_PARAM_BLD(3) for
easier handling of the OSSL_PARAM array.

Fixes #12388

@levitte levitte added branch: master Merge to master branch approval: review pending This pull request needs review by a committer labels Jul 7, 2020
@openssl-machine
Copy link
Collaborator

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

The example to create an EC key from user data didn't show what one
could expect and application to do, especially with regard to how it's
done with raw EC functions.  We therefore refactor it to make proper
use of a BIGNUM where expected, and also use OSSL_PARAM_BLD(3) for
easier handling of the OSSL_PARAM array.

Fixes openssl#12388
@levitte
Copy link
Member Author

levitte commented Sep 5, 2020

Rebased

@paulidale paulidale added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Sep 5, 2020
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Sep 6, 2020
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@levitte
Copy link
Member Author

levitte commented Sep 6, 2020

Merged

d9ea62c DOC: Modify one example in EVP_PKEY_fromdata(3)
6353507 DOC: Fix check of EVP_PKEY_fromdata{,_init} in examples

@levitte levitte closed this Sep 6, 2020
openssl-machine pushed a commit that referenced this pull request Sep 6, 2020
The example to create an EC key from user data didn't show what one
could expect and application to do, especially with regard to how it's
done with raw EC functions.  We therefore refactor it to make proper
use of a BIGNUM where expected, and also use OSSL_PARAM_BLD(3) for
easier handling of the OSSL_PARAM array.

Fixes #12388

Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #12389)
openssl-machine pushed a commit that referenced this pull request Sep 6, 2020
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #12389)
@levitte levitte deleted the fix-12388 branch February 3, 2021 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EVP_PKEY_fromdata/OSSL_PARAM_BN requires data in both big and little endian
5 participants