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

Check for EVP_PKEY_get1_EC_KEY() #484

Closed
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@jer-gentoo

jer-gentoo commented Jul 31, 2016

Second try.

Some distributions patch out EC support because of some patents, so check before use.

jer-gentoo added some commits Jul 30, 2016

Check for EVP_PKEY_get1_EC_KEY()
Some distributions patch out EC support because of some patents, so check before use.
Check for HAVE_EVP_PKEY_EC
Some distributions patch out EC support because of some patents, so check before use.
@ghost

This comment has been minimized.

ghost commented Aug 2, 2016

Hello @jer-gentoo, I just finished working on OpenSSL 1.1.0-pre5, so @dmiller-nmap explained me what was intended here, and asked me if I could merge your work. Thus, I am writing to you to tell you that I've just made the corresponding changes to check for EVP_PKEY_get1_EC_KEY() usability. This has been commited to the SVN trunk in revision 36069.
The modified files are:

M       configure
M       configure.ac
M       nmap_config.h.in
M       nse_ssl_cert.cc

To be short, I've used your configure.ac file as is, autoconf (updated) the configure file, changed HAVE_EVP_PKEY_EC to HAVE_OPENSSL_EC, and declared the latter in nmap_config.h.in.

Of course, your name is mentioned in the commit title.
Thanks for your contribution!
Cheers

@dmiller-nmap

This comment has been minimized.

dmiller-nmap commented Aug 30, 2016

Merged in r36069 (a2b5a1a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment