Fixes some memory leaks and consequent incorrect frees.

5 files changed, 24 insertions, 5 deletions

diff --git a/libcmix-crypto/api.h b/libcmix-crypto/api.h
index f50cee0..8c40d23 100644
--- a/libcmix-crypto/api.h
+++ b/libcmix-crypto/api.h
@@ -53,6 +53,11 @@ typedef void(*ElementToArray)(unsigned char**, size_t* size, GroupElement);
 /*!
  *
  */
+typedef void(*BufferDeleter)(void*);
+
+/*!
+ *
+ */
 typedef GroupElement(*ArrayToElement)(char const*, size_t size, bool);
 
 /*!
@@ -93,6 +98,7 @@ struct Api {
 	KeyPairCreator create_key_pair; ///< Pointer to keypair creation function
 	KeyPairDeleter free_key_pair; ///< Pointer to keypair deletor function
 	ElementToArray element_to_array; ///< Get the array representation of a public key
+	BufferDeleter free_buffer; ///< frees library allocated buffers.
 	ArrayToElement array_to_element; ///< The the GroupElement representation of this array;
 	GroupElementGetter get_group_element; ///< get group element
 	GroupElementDeleter free_group_element; ///< frees a base type of the cryptolibrary.
diff --git a/libcmix-crypto/curve25519/curve25519.c b/libcmix-crypto/curve25519/curve25519.c
index 1a83308..6eaaafc 100644
--- a/libcmix-crypto/curve25519/curve25519.c
+++ b/libcmix-crypto/curve25519/curve25519.c
@@ -14,6 +14,7 @@ struct Api get_curve25519_implementation()
 		NULL,
 		NULL,
 		NULL,
+		NULL,
 		&curve25519_add_public_share,
 		&curve25519_derive_shared_key,
 		&curve25519_shared_key_deleter,
diff --git a/libcmix-crypto/elgamal/elgamal.c b/libcmix-crypto/elgamal/elgamal.c
index 3ee86d7..336d974 100644
--- a/libcmix-crypto/elgamal/elgamal.c
+++ b/libcmix-crypto/elgamal/elgamal.c
@@ -8,6 +8,7 @@ struct Api get_elgamal_implementation()
 		elgamal_create_keypair,
 		elgamal_delete_keypair,
 		elgamal_element_to_array,
+		elgamal_free_buffer,
 		elgamal_array_to_element,
 		elgamal_get_group_element,
 		elgamal_delete_group_element,
diff --git a/libcmix-crypto/elgamal/elgamal.h b/libcmix-crypto/elgamal/elgamal.h
index 746b726..71046c5 100644
--- a/libcmix-crypto/elgamal/elgamal.h
+++ b/libcmix-crypto/elgamal/elgamal.h
@@ -18,6 +18,8 @@ extern KeyPairDeleter elgamal_delete_keypair;
 
 extern ElementToArray elgamal_element_to_array;
 
+extern BufferDeleter elgamal_free_buffer;
+
 extern ArrayToElement elgamal_array_to_element;
 
 extern GroupElementGetter elgamal_get_group_element;
diff --git a/libcmix-crypto/elgamal/gcrypt/gcrypt_elgamal.c b/libcmix-crypto/elgamal/gcrypt/gcrypt_elgamal.c
index 1dd0690..a0512dd 100644
--- a/libcmix-crypto/elgamal/gcrypt/gcrypt_elgamal.c
+++ b/libcmix-crypto/elgamal/gcrypt/gcrypt_elgamal.c
@@ -67,7 +67,7 @@ struct KeyPair gcrypt_elgamal_create_keypair() {
 	gcry_error_t error;
 	
 	void* bytes = gcry_random_bytes_secure(2048/8, GCRY_VERY_STRONG_RANDOM);
-	gcry_mpi_t x = gcry_mpi_snew(2048);
+	gcry_mpi_t x;
 	error = gcry_mpi_scan(&x, GCRYMPI_FMT_USG, bytes, 2048/8, &parse_error_offset);
 	check(error);
 	
@@ -105,12 +105,17 @@ void gcrypt_elgamal_element_to_array(unsigned char** buffer, size_t* len, void*
 	check(error);
 }
 
+void gcrypt_elgamal_free_buffer(void* buffer) {
+	gcry_free(buffer);
+}
+
 void* gcrypt_elgamal_array_to_element(char const* buffer, size_t len, bool secure) {
 	size_t error_pos;
 	gcry_error_t error;
 	
-	gcry_mpi_t mpi = secure ? gcry_mpi_snew(0) : gcry_mpi_new(0);
+	gcry_mpi_t mpi;
 	error = gcry_mpi_scan(&mpi, GCRYMPI_FMT_USG, buffer, len, &error_pos);
+	gcry_mpi_set_flag(mpi, GCRYMPI_FLAG_SECURE);
 	check(error);
 	
 	return mpi;
@@ -121,8 +126,10 @@ void* gcrypt_elgamal_get_group_element(bool secure) {
 	gcry_error_t error;
 	
 	void* bytes = gcry_random_bytes_secure(2048/8, GCRY_VERY_STRONG_RANDOM);
-	gcry_mpi_t a = secure ? gcry_mpi_snew(2048) : gcry_mpi_new(2048);
+	gcry_mpi_t a;
 	error = gcry_mpi_scan(&a, GCRYMPI_FMT_USG, bytes, 2048/8, &parse_error_offset);
+	//random bytes generated with GCRY{,_VERY}_STRONG_RANDOM are generated in "secure memory"
+	//so secure is unused.
 	check(error);
 	
 	gcry_free(bytes);
@@ -149,8 +156,6 @@ void gcrypt_elgamal_delete_group_element(void* el) {
 	gcry_mpi_release((gcry_mpi_t) el);
 }
 
-
-
 void gcrypt_elgamal_add_public_share(char** buffer, size_t* out_len, char const* share, size_t in_len, void* pubkey) {
 	
 	gcry_error_t error;
@@ -167,6 +172,9 @@ void gcrypt_elgamal_add_public_share(char** buffer, size_t* out_len, char const*
 	
 	error = gcry_mpi_aprint(GCRYMPI_FMT_USG, (unsigned char**) buffer, out_len, result);
 	check(error);
+	
+	gcry_mpi_release(mpi_share);
+	gcry_mpi_release(result);
 }
 
 struct SharedKey gcrypt_elgamal_derive_shared_key(struct KeyPair keypair, unsigned char const* other_pub, size_t pub_len, unsigned char const* value, size_t value_len, void* priv_value, bool swap) {
@@ -227,6 +235,7 @@ Initializer elgamal_initialize = &gcrypt_elgamal_initialize;
 KeyPairCreator elgamal_create_keypair = &gcrypt_elgamal_create_keypair;
 KeyPairDeleter elgamal_delete_keypair = &gcrypt_elgamal_delete_keypair;
 ElementToArray elgamal_element_to_array = &gcrypt_elgamal_element_to_array;
+BufferDeleter elgamal_free_buffer = &gcrypt_elgamal_free_buffer;
 ArrayToElement elgamal_array_to_element = &gcrypt_elgamal_array_to_element;
 GroupElementGetter elgamal_get_group_element = &gcrypt_elgamal_get_group_element;
 GroupElementDeleter elgamal_delete_group_element = &gcrypt_elgamal_delete_group_element;