X-Git-Url: https://tinc-vpn.org/git/browse?p=tinc;a=blobdiff_plain;f=src%2Fopenssl%2Fcipher.c;h=d51ec0d5cedd696b75a2d47cdad8472163d08642;hp=d5cafa4197db3d2933d7868327238efe9887837f;hb=f6e87ab476a0faf8b124ecaaa27f967d825e6457;hpb=3a316823b971396a428f020f401b9fe41252d98d

diff --git a/src/openssl/cipher.c b/src/openssl/cipher.c
index d5cafa41..d51ec0d5 100644
--- a/src/openssl/cipher.c
+++ b/src/openssl/cipher.c
@@ -36,14 +36,17 @@ static cipher_t *cipher_open(const EVP_CIPHER *evp_cipher) {
 	cipher_t *cipher = xzalloc(sizeof(*cipher));
 	cipher->cipher = evp_cipher;
 	cipher->ctx = EVP_CIPHER_CTX_new();
-	if(!cipher->ctx)
+
+	if(!cipher->ctx) {
 		abort();
+	}
 
 	return cipher;
 }
 
 cipher_t *cipher_open_by_name(const char *name) {
 	const EVP_CIPHER *evp_cipher = EVP_get_cipherbyname(name);
+
 	if(!evp_cipher) {
 		logger(DEBUG_ALWAYS, LOG_ERR, "Unknown cipher name '%s'!", name);
 		return NULL;
@@ -54,6 +57,7 @@ cipher_t *cipher_open_by_name(const char *name) {
 
 cipher_t *cipher_open_by_nid(int nid) {
 	const EVP_CIPHER *evp_cipher = EVP_get_cipherbynid(nid);
+
 	if(!evp_cipher) {
 		logger(DEBUG_ALWAYS, LOG_ERR, "Unknown cipher nid %d!", nid);
 		return NULL;
@@ -63,16 +67,18 @@ cipher_t *cipher_open_by_nid(int nid) {
 }
 
 void cipher_close(cipher_t *cipher) {
-	if(!cipher)
+	if(!cipher) {
 		return;
+	}
 
 	EVP_CIPHER_CTX_free(cipher->ctx);
 	free(cipher);
 }
 
 size_t cipher_keylength(const cipher_t *cipher) {
-	if(!cipher || !cipher->cipher)
+	if(!cipher || !cipher->cipher) {
 		return 0;
+	}
 
 	return EVP_CIPHER_key_length(cipher->cipher) + EVP_CIPHER_iv_length(cipher->cipher);
 }
@@ -85,8 +91,9 @@ uint64_t cipher_budget(const cipher_t *cipher) {
 	   is limited to what can be represented with a 64 bits integer.
 	 */
 
-	if(!cipher || !cipher->cipher)
-		return UINT64_MAX; // NULL cipher
+	if(!cipher || !cipher->cipher) {
+		return UINT64_MAX;        // NULL cipher
+	}
 
 	int ivlen = EVP_CIPHER_iv_length(cipher->cipher);
 	int blklen = EVP_CIPHER_block_size(cipher->cipher);
@@ -96,8 +103,9 @@ uint64_t cipher_budget(const cipher_t *cipher) {
 }
 
 size_t cipher_blocksize(const cipher_t *cipher) {
-	if(!cipher || !cipher->cipher)
+	if(!cipher || !cipher->cipher) {
 		return 1;
+	}
 
 	return EVP_CIPHER_block_size(cipher->cipher);
 }
@@ -105,13 +113,15 @@ size_t cipher_blocksize(const cipher_t *cipher) {
 bool cipher_set_key(cipher_t *cipher, void *key, bool encrypt) {
 	bool result;
 
-	if(encrypt)
+	if(encrypt) {
 		result = EVP_EncryptInit_ex(cipher->ctx, cipher->cipher, NULL, (unsigned char *)key, (unsigned char *)key + EVP_CIPHER_key_length(cipher->cipher));
-	else
+	} else {
 		result = EVP_DecryptInit_ex(cipher->ctx, cipher->cipher, NULL, (unsigned char *)key, (unsigned char *)key + EVP_CIPHER_key_length(cipher->cipher));
+	}
 
-	if(result)
+	if(result) {
 		return true;
+	}
 
 	logger(DEBUG_ALWAYS, LOG_ERR, "Error while setting key: %s", ERR_error_string(ERR_get_error(), NULL));
 	return false;
@@ -120,13 +130,15 @@ bool cipher_set_key(cipher_t *cipher, void *key, bool encrypt) {
 bool cipher_set_key_from_rsa(cipher_t *cipher, void *key, size_t len, bool encrypt) {
 	bool result;
 
-	if(encrypt)
+	if(encrypt) {
 		result = EVP_EncryptInit_ex(cipher->ctx, cipher->cipher, NULL, (unsigned char *)key + len - EVP_CIPHER_key_length(cipher->cipher), (unsigned char *)key + len - EVP_CIPHER_iv_length(cipher->cipher) - EVP_CIPHER_key_length(cipher->cipher));
-	else
+	} else {
 		result = EVP_DecryptInit_ex(cipher->ctx, cipher->cipher, NULL, (unsigned char *)key + len - EVP_CIPHER_key_length(cipher->cipher), (unsigned char *)key + len - EVP_CIPHER_iv_length(cipher->cipher) - EVP_CIPHER_key_length(cipher->cipher));
+	}
 
-	if(result)
+	if(result) {
 		return true;
+	}
 
 	logger(DEBUG_ALWAYS, LOG_ERR, "Error while setting key: %s", ERR_error_string(ERR_get_error(), NULL));
 	return false;
@@ -135,16 +147,24 @@ bool cipher_set_key_from_rsa(cipher_t *cipher, void *key, size_t len, bool encry
 bool cipher_encrypt(cipher_t *cipher, const void *indata, size_t inlen, void *outdata, size_t *outlen, bool oneshot) {
 	if(oneshot) {
 		int len, pad;
+
 		if(EVP_EncryptInit_ex(cipher->ctx, NULL, NULL, NULL, NULL)
-				&& EVP_EncryptUpdate(cipher->ctx, (unsigned char *)outdata, &len, indata, inlen)
-				&& EVP_EncryptFinal_ex(cipher->ctx, (unsigned char *)outdata + len, &pad)) {
-			if(outlen) *outlen = len + pad;
+		                && EVP_EncryptUpdate(cipher->ctx, (unsigned char *)outdata, &len, indata, inlen)
+		                && EVP_EncryptFinal_ex(cipher->ctx, (unsigned char *)outdata + len, &pad)) {
+			if(outlen) {
+				*outlen = len + pad;
+			}
+
 			return true;
 		}
 	} else {
 		int len;
+
 		if(EVP_EncryptUpdate(cipher->ctx, outdata, &len, indata, inlen)) {
-			if(outlen) *outlen = len;
+			if(outlen) {
+				*outlen = len;
+			}
+
 			return true;
 		}
 	}
@@ -156,16 +176,24 @@ bool cipher_encrypt(cipher_t *cipher, const void *indata, size_t inlen, void *ou
 bool cipher_decrypt(cipher_t *cipher, const void *indata, size_t inlen, void *outdata, size_t *outlen, bool oneshot) {
 	if(oneshot) {
 		int len, pad;
+
 		if(EVP_DecryptInit_ex(cipher->ctx, NULL, NULL, NULL, NULL)
-				&& EVP_DecryptUpdate(cipher->ctx, (unsigned char *)outdata, &len, indata, inlen)
-				&& EVP_DecryptFinal_ex(cipher->ctx, (unsigned char *)outdata + len, &pad)) {
-			if(outlen) *outlen = len + pad;
+		                && EVP_DecryptUpdate(cipher->ctx, (unsigned char *)outdata, &len, indata, inlen)
+		                && EVP_DecryptFinal_ex(cipher->ctx, (unsigned char *)outdata + len, &pad)) {
+			if(outlen) {
+				*outlen = len + pad;
+			}
+
 			return true;
 		}
 	} else {
 		int len;
+
 		if(EVP_EncryptUpdate(cipher->ctx, outdata, &len, indata, inlen)) {
-			if(outlen) *outlen = len;
+			if(outlen) {
+				*outlen = len;
+			}
+
 			return true;
 		}
 	}
@@ -175,8 +203,9 @@ bool cipher_decrypt(cipher_t *cipher, const void *indata, size_t inlen, void *ou
 }
 
 int cipher_get_nid(const cipher_t *cipher) {
-	if(!cipher || !cipher->cipher)
+	if(!cipher || !cipher->cipher) {
 		return 0;
+	}
 
 	return EVP_CIPHER_nid(cipher->cipher);
 }