disable algorithm 13 and 14 in the opensslsigner if OpenSSL does not … #3851

Merged
merged 2 commits into from May 23, 2016

Projects

None yet

3 participants

@mind04
Contributor
mind04 commented May 11, 2016

…support ecdsa or the required curves

@pieterlexis pieterlexis commented on an outdated diff May 11, 2016
m4/ax_check_openssl.m4
@@ -109,6 +109,18 @@ AC_DEFUN([AX_CHECK_OPENSSL], [
[AC_LANG_PROGRAM([#include <openssl/crypto.h>], [CRYPTO_free(NULL)])],
[
AC_MSG_RESULT([yes])
+ openssl_ecdsa=yes
+ AC_CHECK_FUNC(ECDSA_do_sign,
+ [
+ AC_CHECK_DECLS([NID_X9_62_prime256v1, NID_secp384r1], [ : ], [ openssl_ecdsa=no ], [AC_INCLUDES_DEFAULT
+#include <openssl/evp.h>
+ ])
+ ], [
+ openssl_ecdsa=no
+ ])
+ AS_IF([test "x$openssl_ecdsa" = "xyes"], [
+ AC_DEFINE([OPENSSL_USE_ECDSA], [1], [define to 1 if OpenSSL ecdsa support is avalable.])
@pieterlexis
pieterlexis May 11, 2016 Member

I would prefer HAVE_OPENSSL_ECDSA as name of the defined variable.

@Habbie Habbie merged commit 2a5fd54 into PowerDNS:master May 23, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mind04 mind04 deleted the mind04:ecdsa2 branch May 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment