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

Adapt DH to use with KEYMGMT #9394

Closed
wants to merge 1 commit into from
Closed

Conversation

levitte
Copy link
Member

@levitte levitte commented Jul 16, 2019

The biggest part in this was to move the key->param builder from EVP
to the DH ASN.1 method, and to implement the KEYMGMT support in the
provider DH.

@levitte levitte added the branch: master Merge to master branch label Jul 16, 2019
@levitte levitte added this to In progress in 3.0 New Core + FIPS via automation Jul 16, 2019
@levitte levitte force-pushed the evp_keymgmt-dh branch 2 times, most recently from 09a9fdb to 7905081 Compare July 16, 2019 11:22
crypto/dh/dh_ameth.c Outdated Show resolved Hide resolved
crypto/evp/exchange.c Outdated Show resolved Hide resolved
3.0 New Core + FIPS automation moved this from In progress to Needs review Jul 16, 2019
@levitte levitte force-pushed the evp_keymgmt-dh branch 4 times, most recently from 464aa2f to 3b0f8a1 Compare July 17, 2019 17:24
@levitte levitte mentioned this pull request Jul 18, 2019
2 tasks
crypto/dh/dh_rfc7919.c Show resolved Hide resolved
providers/common/exchange/dh.c Outdated Show resolved Hide resolved
providers/common/exchange/dh.c Outdated Show resolved Hide resolved
providers/common/exchange/dh.c Outdated Show resolved Hide resolved
@levitte levitte force-pushed the evp_keymgmt-dh branch 3 times, most recently from bb7fd1e to e224638 Compare July 22, 2019 04:28
@levitte
Copy link
Member Author

levitte commented Jul 22, 2019

Now that #9312 has been merged, it's time to take this out of WIP

@levitte levitte marked this pull request as ready for review July 22, 2019 04:29
@levitte levitte changed the title WIP: Adapt DH to use with KEYMGMT Adapt DH to use with KEYMGMT Jul 22, 2019
@mattcaswell
Copy link
Member

Needs a rebase

@levitte
Copy link
Member Author

levitte commented Jul 22, 2019

Rebase done

@levitte
Copy link
Member Author

levitte commented Jul 22, 2019

Rebased, again

crypto/dh/dh_ameth.c Outdated Show resolved Hide resolved
crypto/dh/dh_ameth.c Outdated Show resolved Hide resolved
Copy link
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved - although needs another rebase!

3.0 New Core + FIPS automation moved this from Needs review to Reviewer approved Jul 23, 2019
@levitte
Copy link
Member Author

levitte commented Jul 23, 2019

Rebased. Only an adjustment of util/libcrypto.num needed, so I count your approval still. Just want to see that the CIs agree, then I'll merge.

@mattcaswell
Copy link
Member

Reconfirm assuming CIs pass.

The biggest part in this was to move the key->param builder from EVP
to the DH ASN.1 method, and to implement the KEYMGMT support in the
provider DH.
@levitte
Copy link
Member Author

levitte commented Jul 23, 2019

An empty line at the end of util/libcrypto.num got the builds to belly up...

@levitte
Copy link
Member Author

levitte commented Jul 23, 2019

Ok! Time to merge

@levitte
Copy link
Member Author

levitte commented Jul 23, 2019

Merged.

8b84b07 Adapt DH to use with KEYMGMT

@levitte levitte closed this Jul 23, 2019
3.0 New Core + FIPS automation moved this from Reviewer approved to Done Jul 23, 2019
levitte added a commit that referenced this pull request Jul 23, 2019
The biggest part in this was to move the key->param builder from EVP
to the DH ASN.1 method, and to implement the KEYMGMT support in the
provider DH.

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #9394)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants