commit c0463502ff89db15c9bcfee33878678b7fd14cef
author Paul Bakker <p.j.bakker@polarssl.org> Thu Feb 14 11:19:38 2013 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Thu Feb 14 11:19:38 2013 +0100
tree aab8052a4b44ed3668ba4da3336d67332075c0dd
parent c7a2da437e5174dd9487c442c6315114c2029169

Fixed memory leak in ssl_free() and ssl_reset() for active session

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index a9654e6..a4c11f6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 PolarSSL ChangeLog
 
+= Master
+Bugfix
+   * Fixed memory leak in ssl_free() and ssl_reset() for active session
+
 = Version 1.2.5 released 2013-02-02
 Changes
    * Allow enabling of dummy error_strerror() to support some use-cases

library/ssl_tls.c

diff --git a/library/ssl_tls.c b/library/ssl_tls.c
index 4194ac0..44cb92b 100644
--- a/library/ssl_tls.c
+++ b/library/ssl_tls.c
@@ -3025,6 +3025,13 @@
         ssl->transform = NULL;
     }
 
+    if( ssl->session )
+    {
+        ssl_session_free( ssl->session );
+        free( ssl->session );
+        ssl->session = NULL;
+    }
+
     if( ( ret = ssl_handshake_init( ssl ) ) != 0 )
         return( ret );
 
@@ -3891,6 +3898,12 @@
         free( ssl->session_negotiate );
     }
 
+    if( ssl->session )
+    {
+        ssl_session_free( ssl->session );
+        free( ssl->session );
+    }
+
     if ( ssl->hostname != NULL)
     {
         memset( ssl->hostname, 0, ssl->hostname_len );