From: Christian Grothoff <christian@grothoff.org>
Date: Thu, 12 Dec 2013 18:38:31 +0000 (+0000)
Subject: -fix leaks of MPIs
X-Git-Tag: initial-import-from-subversion-38251~5519
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=95b1e48893a606baa3b6dd78eee8c2dc1bb25400;p=oweals%2Fgnunet.git

-fix leaks of MPIs
---

diff --git a/src/secretsharing/gnunet-service-secretsharing.c b/src/secretsharing/gnunet-service-secretsharing.c
index 926861ac7..78ff4f828 100644
--- a/src/secretsharing/gnunet-service-secretsharing.c
+++ b/src/secretsharing/gnunet-service-secretsharing.c
@@ -558,6 +558,7 @@ insert_round2_element (struct KeygenSession *ks)
   gcry_mpi_t v;
 
   GNUNET_assert (0 != (c = gcry_mpi_new (PAILLIER_BITS)));
+  // FIXME: c is never used...
   GNUNET_assert (0 != (v = gcry_mpi_new (PAILLIER_BITS)));
   GNUNET_assert (0 != (idx = gcry_mpi_new (PAILLIER_BITS)));
 
@@ -611,6 +612,10 @@ insert_round2_element (struct KeygenSession *ks)
 
   GNUNET_CONSENSUS_insert (ks->consensus, element, NULL, NULL);
   GNUNET_free (element); /* FIXME: maybe stack-allocate instead? */
+
+  gcry_mpi_release (c);
+  gcry_mpi_release (v);
+  gcry_mpi_release (idx);
 }