]> git.ipfire.org Git - thirdparty/strongswan.git/commitdiff
openssl: Move ENGINE-specific code into a separate file
authorTobias Brunner <tobias@strongswan.org>
Fri, 4 Mar 2022 10:26:52 +0000 (11:26 +0100)
committerTobias Brunner <tobias@strongswan.org>
Thu, 14 Apr 2022 17:05:44 +0000 (19:05 +0200)
This way we can compile it with OPENSSL_SUPPRESS_DEPRECATED for
OpenSSL 3.0, which deprecated the ENGINE API.

src/libstrongswan/plugins/openssl/Makefile.am
src/libstrongswan/plugins/openssl/openssl_engine.c [new file with mode: 0644]
src/libstrongswan/plugins/openssl/openssl_engine.h [new file with mode: 0644]
src/libstrongswan/plugins/openssl/openssl_plugin.c

index 4df4a81b417c5416fc06c529cb4f3e48b40a2cb0..979bd88861388ca4c95d6ad8c452bfd54eda001c 100644 (file)
@@ -19,6 +19,7 @@ libstrongswan_openssl_la_SOURCES = \
        openssl_plugin.h openssl_plugin.c \
        openssl_util.c openssl_util.h \
        openssl_crypter.c openssl_crypter.h \
+       openssl_engine.c openssl_engine.h \
        openssl_hasher.c openssl_hasher.h \
        openssl_sha1_prf.c openssl_sha1_prf.h \
        openssl_diffie_hellman.c openssl_diffie_hellman.h \
diff --git a/src/libstrongswan/plugins/openssl/openssl_engine.c b/src/libstrongswan/plugins/openssl/openssl_engine.c
new file mode 100644 (file)
index 0000000..a4b2cba
--- /dev/null
@@ -0,0 +1,214 @@
+/*
+ * Copyright (C) 2008-2018 Tobias Brunner
+ * Copyright (C) 2008 Martin Willi
+ * HSR Hochschule fuer Technik Rapperswil
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.  See <http://www.fsf.org/copyleft/gpl.txt>.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * for more details.
+ */
+
+/* the ENGINE API has been deprecated with OpenSSL 3.0 (the provider API should
+ * be used instead) */
+#define OPENSSL_SUPPRESS_DEPRECATED
+
+#include <openssl/opensslv.h>
+#include <openssl/opensslconf.h>
+
+#include "openssl_engine.h"
+
+#if !defined(OPENSSL_NO_ENGINE) && \
+       (OPENSSL_VERSION_NUMBER < 0x30000000L || !defined(OPENSSL_NO_DEPRECATED))
+
+#include <openssl/engine.h>
+
+#include "openssl_ec_private_key.h"
+#include "openssl_ed_private_key.h"
+#include "openssl_rsa_private_key.h"
+
+/**
+ * Login to engine with a PIN specified for a keyid
+ */
+static bool login(ENGINE *engine, chunk_t keyid)
+{
+       enumerator_t *enumerator;
+       shared_key_t *shared;
+       identification_t *id;
+       chunk_t key;
+       char pin[64];
+       bool found = FALSE, success = FALSE;
+
+       id = identification_create_from_encoding(ID_KEY_ID, keyid);
+       enumerator = lib->credmgr->create_shared_enumerator(lib->credmgr,
+                                                                                                               SHARED_PIN, id, NULL);
+       while (enumerator->enumerate(enumerator, &shared, NULL, NULL))
+       {
+               found = TRUE;
+               key = shared->get_key(shared);
+               if (snprintf(pin, sizeof(pin),
+                                        "%.*s", (int)key.len, key.ptr) >= sizeof(pin))
+               {
+                       continue;
+               }
+               if (ENGINE_ctrl_cmd_string(engine, "PIN", pin, 0))
+               {
+                       success = TRUE;
+                       break;
+               }
+               else
+               {
+                       DBG1(DBG_CFG, "setting PIN on engine failed");
+               }
+       }
+       enumerator->destroy(enumerator);
+       id->destroy(id);
+       if (!found)
+       {
+               DBG1(DBG_CFG, "no PIN found for %#B", &keyid);
+       }
+       return success;
+}
+
+/*
+ * Described in header
+ */
+private_key_t *openssl_private_key_connect(key_type_t type, va_list args)
+{
+       char *engine_id = NULL;
+       char keyname[BUF_LEN];
+       chunk_t keyid = chunk_empty;
+       EVP_PKEY *key;
+       ENGINE *engine;
+       int slot = -1;
+
+       while (TRUE)
+       {
+               switch (va_arg(args, builder_part_t))
+               {
+                       case BUILD_PKCS11_KEYID:
+                               keyid = va_arg(args, chunk_t);
+                               continue;
+                       case BUILD_PKCS11_SLOT:
+                               slot = va_arg(args, int);
+                               continue;
+                       case BUILD_PKCS11_MODULE:
+                               engine_id = va_arg(args, char*);
+                               continue;
+                       case BUILD_END:
+                               break;
+                       default:
+                               return NULL;
+               }
+               break;
+       }
+       if (!keyid.len)
+       {
+               return NULL;
+       }
+
+       memset(keyname, 0, sizeof(keyname));
+       if (slot != -1)
+       {
+               snprintf(keyname, sizeof(keyname), "%d:", slot);
+       }
+       if (sizeof(keyname) - strlen(keyname) <= keyid.len * 2 + 1)
+       {
+               return NULL;
+       }
+       chunk_to_hex(keyid, keyname + strlen(keyname), FALSE);
+
+       if (!engine_id)
+       {
+               engine_id = lib->settings->get_str(lib->settings,
+                                                       "%s.plugins.openssl.engine_id", "pkcs11", lib->ns);
+       }
+       engine = ENGINE_by_id(engine_id);
+       if (!engine)
+       {
+               DBG2(DBG_LIB, "engine '%s' is not available", engine_id);
+               return NULL;
+       }
+       if (!ENGINE_init(engine))
+       {
+               DBG1(DBG_LIB, "failed to initialize engine '%s'", engine_id);
+               ENGINE_free(engine);
+               return NULL;
+       }
+       ENGINE_free(engine);
+       if (!login(engine, keyid))
+       {
+               DBG1(DBG_LIB, "login to engine '%s' failed", engine_id);
+               ENGINE_finish(engine);
+               return NULL;
+       }
+       key = ENGINE_load_private_key(engine, keyname, NULL, NULL);
+       ENGINE_finish(engine);
+       if (!key)
+       {
+               DBG1(DBG_LIB, "failed to load private key with ID '%s' from "
+                        "engine '%s'", keyname, engine_id);
+               return NULL;
+       }
+
+       switch (EVP_PKEY_base_id(key))
+       {
+#ifndef OPENSSL_NO_RSA
+               case EVP_PKEY_RSA:
+                       return openssl_rsa_private_key_create(key, TRUE);
+#endif
+#ifndef OPENSSL_NO_ECDSA
+               case EVP_PKEY_EC:
+                       return openssl_ec_private_key_create(key, TRUE);
+#endif
+#if OPENSSL_VERSION_NUMBER >= 0x1010100fL && !defined(OPENSSL_NO_EC)
+               case EVP_PKEY_ED25519:
+               case EVP_PKEY_ED448:
+                       return openssl_ed_private_key_create(key, TRUE);
+#endif /* OPENSSL_VERSION_NUMBER */
+               default:
+                       EVP_PKEY_free(key);
+                       break;
+       }
+       return NULL;
+}
+
+/*
+ * Described in header
+ */
+void openssl_engine_deinit()
+{
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
+       ENGINE_cleanup();
+#endif
+}
+
+/*
+ * Described in header
+ */
+void openssl_engine_init()
+{
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
+       /* activate support for hardware accelerators */
+       ENGINE_load_builtin_engines();
+       ENGINE_register_all_complete();
+#endif
+}
+
+#else /* OPENSSL_NO_ENGINE */
+
+private_key_t *openssl_private_key_connect(key_type_t type, va_list args)
+{
+       return NULL;
+}
+
+void openssl_engine_deinit() {}
+
+void openssl_engine_init() {}
+
+#endif /* OPENSSL_NO_ENGINE */
diff --git a/src/libstrongswan/plugins/openssl/openssl_engine.h b/src/libstrongswan/plugins/openssl/openssl_engine.h
new file mode 100644 (file)
index 0000000..d25d123
--- /dev/null
@@ -0,0 +1,53 @@
+/*
+ * Copyright (C) 2022 Tobias Brunner, codelabs GmbH
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
+/**
+ * Compatibility code for legacy ENGINE support.
+ *
+ * @defgroup openssl_engine openssl_engine
+ * @{ @ingroup openssl_p
+ */
+
+#ifndef OPENSSL_ENGINE_H_
+#define OPENSSL_ENGINE_H_
+
+#include <credentials/keys/private_key.h>
+
+/**
+ * Load a private key from a token/ENGINE.
+ *
+ * @param type         key type to load
+ * @param args         build arguments
+ */
+private_key_t *openssl_private_key_connect(key_type_t type, va_list args);
+
+/**
+ * Initialize ENGINE support.
+ */
+void openssl_engine_init();
+
+/**
+ * Deinitialize ENGINE support.
+ */
+void openssl_engine_deinit();
+
+#endif /** OPENSSL_ENGINE_H_ @}*/
index 3446dd0d4933c1a596ffb2eba6fc6c36976b9476..77422f8bed10a83409032f70baa5bcabb9050c6d 100644 (file)
@@ -25,9 +25,6 @@
 #include <openssl/conf.h>
 #include <openssl/rand.h>
 #include <openssl/crypto.h>
-#ifndef OPENSSL_NO_ENGINE
-#include <openssl/engine.h>
-#endif
 #ifndef OPENSSL_NO_ECDH
 #include <openssl/ec.h>
 #endif
@@ -38,6 +35,7 @@
 #include "openssl_plugin.h"
 #include "openssl_util.h"
 #include "openssl_crypter.h"
+#include "openssl_engine.h"
 #include "openssl_hasher.h"
 #include "openssl_sha1_prf.h"
 #include "openssl_diffie_hellman.h"
@@ -316,157 +314,6 @@ static private_key_t *openssl_private_key_load(key_type_t type, va_list args)
        return NULL;
 }
 
-#ifndef OPENSSL_NO_ENGINE
-/**
- * Login to engine with a PIN specified for a keyid
- */
-static bool login(ENGINE *engine, chunk_t keyid)
-{
-       enumerator_t *enumerator;
-       shared_key_t *shared;
-       identification_t *id;
-       chunk_t key;
-       char pin[64];
-       bool found = FALSE, success = FALSE;
-
-       id = identification_create_from_encoding(ID_KEY_ID, keyid);
-       enumerator = lib->credmgr->create_shared_enumerator(lib->credmgr,
-                                                                                                               SHARED_PIN, id, NULL);
-       while (enumerator->enumerate(enumerator, &shared, NULL, NULL))
-       {
-               found = TRUE;
-               key = shared->get_key(shared);
-               if (snprintf(pin, sizeof(pin),
-                                        "%.*s", (int)key.len, key.ptr) >= sizeof(pin))
-               {
-                       continue;
-               }
-               if (ENGINE_ctrl_cmd_string(engine, "PIN", pin, 0))
-               {
-                       success = TRUE;
-                       break;
-               }
-               else
-               {
-                       DBG1(DBG_CFG, "setting PIN on engine failed");
-               }
-       }
-       enumerator->destroy(enumerator);
-       id->destroy(id);
-       if (!found)
-       {
-               DBG1(DBG_CFG, "no PIN found for %#B", &keyid);
-       }
-       return success;
-}
-#endif /* OPENSSL_NO_ENGINE */
-
-/**
- * Load private key via engine
- */
-static private_key_t *openssl_private_key_connect(key_type_t type,
-                                                                                                 va_list args)
-{
-#ifndef OPENSSL_NO_ENGINE
-       char *engine_id = NULL;
-       char keyname[BUF_LEN];
-       chunk_t keyid = chunk_empty;
-       EVP_PKEY *key;
-       ENGINE *engine;
-       int slot = -1;
-
-       while (TRUE)
-       {
-               switch (va_arg(args, builder_part_t))
-               {
-                       case BUILD_PKCS11_KEYID:
-                               keyid = va_arg(args, chunk_t);
-                               continue;
-                       case BUILD_PKCS11_SLOT:
-                               slot = va_arg(args, int);
-                               continue;
-                       case BUILD_PKCS11_MODULE:
-                               engine_id = va_arg(args, char*);
-                               continue;
-                       case BUILD_END:
-                               break;
-                       default:
-                               return NULL;
-               }
-               break;
-       }
-       if (!keyid.len)
-       {
-               return NULL;
-       }
-
-       memset(keyname, 0, sizeof(keyname));
-       if (slot != -1)
-       {
-               snprintf(keyname, sizeof(keyname), "%d:", slot);
-       }
-       if (sizeof(keyname) - strlen(keyname) <= keyid.len * 2 + 1)
-       {
-               return NULL;
-       }
-       chunk_to_hex(keyid, keyname + strlen(keyname), FALSE);
-
-       if (!engine_id)
-       {
-               engine_id = lib->settings->get_str(lib->settings,
-                                                       "%s.plugins.openssl.engine_id", "pkcs11", lib->ns);
-       }
-       engine = ENGINE_by_id(engine_id);
-       if (!engine)
-       {
-               DBG2(DBG_LIB, "engine '%s' is not available", engine_id);
-               return NULL;
-       }
-       if (!ENGINE_init(engine))
-       {
-               DBG1(DBG_LIB, "failed to initialize engine '%s'", engine_id);
-               ENGINE_free(engine);
-               return NULL;
-       }
-       ENGINE_free(engine);
-       if (!login(engine, keyid))
-       {
-               DBG1(DBG_LIB, "login to engine '%s' failed", engine_id);
-               ENGINE_finish(engine);
-               return NULL;
-       }
-       key = ENGINE_load_private_key(engine, keyname, NULL, NULL);
-       ENGINE_finish(engine);
-       if (!key)
-       {
-               DBG1(DBG_LIB, "failed to load private key with ID '%s' from "
-                        "engine '%s'", keyname, engine_id);
-               return NULL;
-       }
-
-       switch (EVP_PKEY_base_id(key))
-       {
-#ifndef OPENSSL_NO_RSA
-               case EVP_PKEY_RSA:
-                       return openssl_rsa_private_key_create(key, TRUE);
-#endif
-#ifndef OPENSSL_NO_ECDSA
-               case EVP_PKEY_EC:
-                       return openssl_ec_private_key_create(key, TRUE);
-#endif
-#if OPENSSL_VERSION_NUMBER >= 0x1010100fL && !defined(OPENSSL_NO_EC)
-               case EVP_PKEY_ED25519:
-               case EVP_PKEY_ED448:
-                       return openssl_ed_private_key_create(key, TRUE);
-#endif /* OPENSSL_VERSION_NUMBER */
-               default:
-                       EVP_PKEY_free(key);
-                       break;
-       }
-#endif /* OPENSSL_NO_ENGINE */
-       return NULL;
-}
-
 METHOD(plugin_t, get_name, char*,
        private_openssl_plugin_t *this)
 {
@@ -865,6 +712,7 @@ METHOD(plugin_t, get_features, int,
 METHOD(plugin_t, destroy, void,
        private_openssl_plugin_t *this)
 {
+
 /* OpenSSL 1.1.0 cleans up itself at exit and while OPENSSL_cleanup() exists we
  * can't call it as we couldn't re-initialize the library (as required by the
  * unit tests and the Android app) */
@@ -874,9 +722,7 @@ METHOD(plugin_t, destroy, void,
        OBJ_cleanup();
 #endif
        EVP_cleanup();
-#ifndef OPENSSL_NO_ENGINE
-       ENGINE_cleanup();
-#endif /* OPENSSL_NO_ENGINE */
+       openssl_engine_deinit();
        CRYPTO_cleanup_all_ex_data();
        threading_cleanup();
        ERR_free_strings();
@@ -960,11 +806,7 @@ plugin_t *openssl_plugin_create()
        OPENSSL_config(NULL);
 #endif
        OpenSSL_add_all_algorithms();
-#ifndef OPENSSL_NO_ENGINE
-       /* activate support for hardware accelerators */
-       ENGINE_load_builtin_engines();
-       ENGINE_register_all_complete();
-#endif /* OPENSSL_NO_ENGINE */
+       openssl_engine_init();
 #endif /* OPENSSL_VERSION_NUMBER */
 
 #if OPENSSL_VERSION_NUMBER >= 0x30000000L