Simplify configuration of cipher and MAC implementations

Let the cipher and MAC handlers just store the chosen implementations themselves
instead of relying on the global configuration.

2 files changed, 17 insertions, 31 deletions

diff --git a/src/crypto/cipher/ciphers.c.in b/src/crypto/cipher/ciphers.c.in
index 1796cc5..01b5a35 100644
--- a/src/crypto/cipher/ciphers.c.in
+++ b/src/crypto/cipher/ciphers.c.in
@@ -46,14 +46,15 @@ typedef struct cipher_entry {
 static const cipher_entry_t ciphers[] = { @CIPHER_LIST@
 };
 
+static const fastd_cipher_t *cipher_conf[array_size(ciphers)] = {};
+
 
 static inline bool cipher_available(const fastd_cipher_t *cipher) {
 	return (!cipher->available) || cipher->available();
 }
 
-const fastd_cipher_t** fastd_cipher_config_alloc(void) {
-	const fastd_cipher_t **cipher_conf = calloc(array_size(ciphers), sizeof(const fastd_cipher_t*));
-
+/** Initializes the list of ciphers */
+void fastd_cipher_init(void) {
 	size_t i, j;
 	for (i = 0; i < array_size(ciphers); i++) {
 		for (j = 0; ciphers[i].impls[j].impl; j++) {
@@ -63,15 +64,10 @@ const fastd_cipher_t** fastd_cipher_config_alloc(void) {
 
 		cipher_conf[i] = ciphers[i].impls[j].impl;
 	}
-
-	return cipher_conf;
-}
-
-void fastd_cipher_config_free(const fastd_cipher_t **cipher_conf) {
-	free(cipher_conf);
 }
 
-bool fastd_cipher_config(const fastd_cipher_t **cipher_conf, const char *name, const char *impl) {
+/** Configures a cipher to use a specific implementation */
+bool fastd_cipher_config(const char *name, const char *impl) {
 	size_t i;
 	for (i = 0; i < array_size(ciphers); i++) {
 		if (!strcmp(ciphers[i].name, name)) {
@@ -94,15 +90,13 @@ bool fastd_cipher_config(const fastd_cipher_t **cipher_conf, const char *name, c
 }
 
 const fastd_cipher_info_t* fastd_cipher_info_get_by_name(const char *name) {
-	size_t i, j;
+	size_t i;
 	for (i = 0; i < array_size(ciphers); i++) {
 		if (strcmp(ciphers[i].name, name))
 			continue;
 
-		for (j = 0; ciphers[i].impls[j].impl; j++) {
-			if (cipher_available(ciphers[i].impls[j].impl))
-				return ciphers[i].info;
-		}
+		if (cipher_conf[i])
+			return ciphers[i].info;
 
 		break;
 	}
@@ -114,7 +108,7 @@ const fastd_cipher_t* fastd_cipher_get(const fastd_cipher_info_t *info) {
 	size_t i;
 	for (i = 0; i < array_size(ciphers); i++) {
 		if (ciphers[i].info == info)
-			return conf.ciphers[i];
+			return cipher_conf[i];
 	}
 
 	return NULL;
diff --git a/src/crypto/mac/macs.c.in b/src/crypto/mac/macs.c.in
index 3a8c943..0db26d9 100644
--- a/src/crypto/mac/macs.c.in
+++ b/src/crypto/mac/macs.c.in
@@ -46,14 +46,14 @@ typedef struct mac_entry {
 static const mac_entry_t macs[] = { @MAC_LIST@
 };
 
+static const fastd_mac_t *mac_conf[array_size(macs)] = {};
+
 
 static inline bool mac_available(const fastd_mac_t *mac) {
 	return (!mac->available) || mac->available();
 }
 
-const fastd_mac_t** fastd_mac_config_alloc(void) {
-	const fastd_mac_t **mac_conf = calloc(array_size(macs), sizeof(const fastd_mac_t*));
-
+void fastd_mac_init(void) {
 	size_t i, j;
 	for (i = 0; i < array_size(macs); i++) {
 		for (j = 0; macs[i].impls[j].impl; j++) {
@@ -63,15 +63,9 @@ const fastd_mac_t** fastd_mac_config_alloc(void) {
 
 		mac_conf[i] = macs[i].impls[j].impl;
 	}
-
-	return mac_conf;
-}
-
-void fastd_mac_config_free(const fastd_mac_t **mac_conf) {
-	free(mac_conf);
 }
 
-bool fastd_mac_config(const fastd_mac_t **mac_conf, const char *name, const char *impl) {
+bool fastd_mac_config(const char *name, const char *impl) {
 	size_t i;
 	for (i = 0; i < array_size(macs); i++) {
 		if (!strcmp(macs[i].name, name)) {
@@ -99,10 +93,8 @@ const fastd_mac_info_t* fastd_mac_info_get_by_name(const char *name) {
 		if (strcmp(macs[i].name, name))
 			continue;
 
-		for (j = 0; macs[i].impls[j].impl; j++) {
-			if (mac_available(macs[i].impls[j].impl))
-				return macs[i].info;
-		}
+		if (mac_conf[i])
+			return macs[i].info;
 
 		break;
 	}
@@ -114,7 +106,7 @@ const fastd_mac_t* fastd_mac_get(const fastd_mac_info_t *info) {
 	size_t i;
 	for (i = 0; i < array_size(macs); i++) {
 		if (macs[i].info == info)
-			return conf.macs[i];
+			return mac_conf[i];
 	}
 
 	return NULL;