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

Check for EVP_PKEY_get1_EC_KEY() #484

Closed
wants to merge 4 commits into from
Closed

Check for EVP_PKEY_get1_EC_KEY() #484

wants to merge 4 commits into from

Conversation

jer-gentoo
Copy link

Second try.

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

jer-gentoo and others added 4 commits July 30, 2016 13:05
Some distributions patch out EC support because of some patents, so check before use.
Some distributions patch out EC support because of some patents, so check before use.
@ghost
Copy link

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
Copy link

Merged in r36069 (a2b5a1a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants