commit 61d113bb7b976d543569b2627fc1906652309067
author Paul Bakker <p.j.bakker@polarssl.org> Thu Jul 04 11:51:43 2013 +0200
committer Paul Bakker <p.j.bakker@polarssl.org> Tue Jul 16 17:48:58 2013 +0200
tree 28f489e268042066c502b4bf810a9012e208f76b
parent 8647eecf9026a0e2cf75d00286362a16eed73fda

Init and free new contexts in the right place for SSL to prevent
memory leaks

library/ssl_srv.c

diff --git a/library/ssl_srv.c b/library/ssl_srv.c
index 451d445..c6a8273 100644
--- a/library/ssl_srv.c
+++ b/library/ssl_srv.c
@@ -1304,7 +1304,6 @@
          *     ECPoint      public;
          * } ServerECDHParams;
          */
-        ecdh_init( &ssl->handshake->ecdh_ctx );
         if( ( ret = ecp_use_known_dp( &ssl->handshake->ecdh_ctx.grp,
                                        ssl->handshake->ec_curve ) ) != 0 )
         {
@@ -1423,6 +1422,13 @@
             md_update( &ctx, ssl->handshake->randbytes, 64 );
             md_update( &ctx, dig_sig, dig_sig_len );
             md_finish( &ctx, hash );
+
+            if( ( ret = md_free_ctx( &ctx ) ) != 0 )
+            {
+                SSL_DEBUG_RET( 1, "md_free_ctx", ret );
+                return( ret );
+            }
+
         }
 
         SSL_DEBUG_BUF( 3, "parameters hash", hash, hashlen );