From a3c28f2558481536c03dcfbc2d3b3a7c925b900f Mon Sep 17 00:00:00 2001
From: Aris Adamantiadis <aris@0xbadc0de.be>
Date: Mon, 13 Jun 2011 14:06:30 +0200
Subject: Fix memory leak

---
 src/wrapper.c | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

(limited to 'src/wrapper.c')

diff --git a/src/wrapper.c b/src/wrapper.c
index 11482f88..5a6ed084 100644
--- a/src/wrapper.c
+++ b/src/wrapper.c
@@ -111,7 +111,8 @@ void crypto_free(struct ssh_crypto_struct *crypto){
   bignum_free(crypto->x);
   bignum_free(crypto->y);
   bignum_free(crypto->k);
-  /* lot of other things */
+  SAFE_FREE(crypto->ecdh_client_pubkey);
+  SAFE_FREE(crypto->ecdh_server_pubkey);
 
 #ifdef WITH_LIBZ
   if (crypto->compress_out_ctx &&
@@ -123,8 +124,23 @@ void crypto_free(struct ssh_crypto_struct *crypto){
     inflateEnd(crypto->compress_in_ctx);
   }
 #endif
+  if(crypto->encryptIV)
+    SAFE_FREE(crypto->encryptIV);
+  if(crypto->decryptIV)
+    SAFE_FREE(crypto->decryptIV);
+  if(crypto->encryptMAC)
+    SAFE_FREE(crypto->encryptMAC);
+  if(crypto->decryptMAC)
+    SAFE_FREE(crypto->decryptMAC);
+  if(crypto->encryptkey){
+    memset(crypto->encryptkey, 0, crypto->digest_len);
+    SAFE_FREE(crypto->encryptkey);
+  }
+  if(crypto->decryptkey){
+    memset(crypto->decryptkey, 0, crypto->digest_len);
+    SAFE_FREE(crypto->decryptkey);
+  }
 
-  /* i'm lost in my own code. good work */
   memset(crypto,0,sizeof(*crypto));
 
   SAFE_FREE(crypto);
-- 
cgit