-
-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Fix d2i_AutoPrivateKey_ex so that is uses the new decoder (and produces non legacy keys). #13591
Conversation
@levitte - this produces an interesting failure in the fuzz tests.. dumpasn1 fuzz/corpora/asn1/2b5bdcbf1810066fcc04831b9b60365150e5340c Basically there are no domain params in this data so when it goes to cache it tries to access dsa->param and falls over.. |
Why doesn't the ci fuzz target complain about the issue? |
Not sure. It was failing locally.. |
I've opened an issue at google/oss-fuzz#4767 |
They are unhappy because of the issue I just reported :) |
CI loop is happy now that the DSA_size() NULL checks have been added. |
This pull request is ready to merge |
Ping |
ping |
ping |
looks like this doesnt build now :) |
73d58d1
to
b3a0fc1
Compare
Rebase and removed duplicate function evp_pkey_type2name. |
non legacy keys). Fixes openssl#13522
b3a0fc1
to
22ed598
Compare
Rebased again.. |
ping |
This pull request is ready to merge |
Thanks for reviewing. Merged to master. |
Fixes #13522
Checklist