Skip to content

Adapt X509_PUBKEY_set() for use with provided implementations#10851

Closed
levitte wants to merge 3 commits intoopenssl:masterfrom
levitte:prov-X509_PUBKEY_set
Closed

Adapt X509_PUBKEY_set() for use with provided implementations#10851
levitte wants to merge 3 commits intoopenssl:masterfrom
levitte:prov-X509_PUBKEY_set

Conversation

@levitte
Copy link
Member

@levitte levitte commented Jan 15, 2020

We do this by letting a serializer serialize the provider side key to
a DER blob formatted according to the SubjectPublicKeyInfo structure
(see RFC 5280), and deserialize it in libcrypto using the usual d2i
function.

@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Jan 15, 2020
@levitte levitte force-pushed the prov-X509_PUBKEY_set branch 2 times, most recently from c9c32fc to 3102cff Compare January 18, 2020 04:53
@levitte levitte force-pushed the prov-X509_PUBKEY_set branch from 3102cff to 85a7657 Compare January 21, 2020 20:17
@levitte
Copy link
Member Author

levitte commented Jan 23, 2020

Ping

1 similar comment
@levitte
Copy link
Member Author

levitte commented Jan 24, 2020

Ping

We do this by letting a serializer serialize the provider side key to
a DER blob formatted according to the SubjectPublicKeyInfo structure
(see RFC 5280), and deserialize it in libcrypto using the usual d2i
function.
@levitte levitte force-pushed the prov-X509_PUBKEY_set branch from ef8cefe to 8ef3323 Compare January 26, 2020 16:22
@mattcaswell mattcaswell 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 Jan 27, 2020
openssl-machine pushed a commit that referenced this pull request Jan 28, 2020
We do this by letting a serializer serialize the provider side key to
a DER blob formatted according to the SubjectPublicKeyInfo structure
(see RFC 5280), and deserialize it in libcrypto using the usual d2i
function.

Reviewed-by: Matt Caswell <[email protected]>
(Merged from #10851)
@levitte
Copy link
Member Author

levitte commented Jan 28, 2020

Merged.

e56ba0e Adapt X509_PUBKEY_set() for use with provided implementations

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 Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants