Skip to content

Finish migrating RSAOpenSsl from RSA* to EVP_PKEY* #54282

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.

Sign up for GitHub

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

Merged
bartonjs merged 18 commits into from
Jul 14, 2021
Merged

Finish migrating RSAOpenSsl from RSA* to EVP_PKEY* #54282

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b91a2f8
Add EVP_PKEY based Decode methods.
bartonjs Jun 10, 2021
2c39b94
Change RSAOpenSsl to use new EVP_PKEY-based imports
bartonjs Jun 10, 2021
d946012
Native methods for EVP_PKEY to PKCS8PrivateKeyInfo
bartonjs Jun 11, 2021
186bf50
Change RSAOpenSsl export methods to be PKCS8-based
bartonjs Jun 11, 2021
a410ffb
Native support for EVP_PKEY to SubjectPublicKeyInfo
bartonjs Jun 11, 2021
64ec3a0
Eliminate SafeRsaHandle from the Unix build
bartonjs Jun 11, 2021
0e35eba
Remove RSA*-based PALEXPORT functions
bartonjs Jun 11, 2021
9e422fb
"Add" deleted file.
bartonjs Jun 11, 2021
2f75b09
Make the S.S.C.OpenSsl library work again
bartonjs Jun 16, 2021
7c5a5f8
Bringing back some small amount of RSA* PALEXPORTs.
bartonjs Jun 16, 2021
df42f68
Undo unintended config change
bartonjs Jun 16, 2021
5528ab3
Fix some build breaks
bartonjs Jun 16, 2021
80f753e
Add support for OpenSSL 1.0.x
bartonjs Jun 17, 2021
64b2f37
Remove PALEXPORT EVP_PKEY get/set RSA functions
bartonjs Jun 17, 2021
6b85d76
Error code, take 3.
bartonjs Jun 17, 2021
6e0444c
Merge remote-tracking branch 'dotnet/main' into finish_rsa_evp_pkey
bartonjs Jul 13, 2021
5900493
Clean up references to renamed functions.
bartonjs Jul 13, 2021
e05907d
Make new code build with OpenSSL 3 headers.
bartonjs Jul 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Make new code build with OpenSSL 3 headers.
  • Loading branch information
@bartonjs
bartonjs committed Jul 13, 2021
commit e05907d7f657eee4896f2e2b1985891f24da3b40
2 changes: 1 addition & 1 deletion src/libraries/Native/Unix/System.Security.Cryptography.Native/apibridge.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -834,7 +834,7 @@ int local_EVP_PKEY_check(EVP_PKEY_CTX* ctx)
{
case NID_rsaEncryption:
{
RSA* rsa = EVP_PKEY_get0_RSA(pkey);
const RSA* rsa = EVP_PKEY_get0_RSA(pkey);

if (rsa != NULL)
{
Expand Down
7 changes: 5 additions & 2 deletions src/libraries/Native/Unix/System.Security.Cryptography.Native/pal_evp_pkey.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,12 @@ EVP_PKEY* CryptoNative_EvpPKeyDuplicate(EVP_PKEY* currentKey, int32_t algId)

if (currentAlgId == EVP_PKEY_RSA)
{
RSA* rsa = EVP_PKEY_get0_RSA(currentKey);
const RSA* rsa = EVP_PKEY_get0_RSA(currentKey);

if (rsa == NULL || !EVP_PKEY_set1_RSA(newKey, rsa))
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wcast-qual"
if (rsa == NULL || !EVP_PKEY_set1_RSA(newKey, (RSA*)rsa))
#pragma clang diagnostic pop
{
success = false;
}
Expand Down