commit a565aceea1d25c94e8f80380f305bf8af4a66d5c
author Paul Bakker <p.j.bakker@polarssl.org> Wed Sep 11 11:45:41 2013 +0200
committer Paul Bakker <p.j.bakker@polarssl.org> Wed Sep 11 13:31:53 2013 +0200
tree 4aff94f9b4721a657214a4b88ca5da217ede77a9
parent 78020fe72cb9d2c08d128fea194162b0e3b5e9da

Fixed potential memory leak when failing to resume a session

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index ae02680..ab851fc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,7 @@
 
 = Branch 1.2
 Bugfix
+   * Fixed potential memory leak when failing to resume a session
    * Minor fixes
 
 Security

library/ssl_tls.c

diff --git a/library/ssl_tls.c b/library/ssl_tls.c
index fd5b5a3..1574217 100644
--- a/library/ssl_tls.c
+++ b/library/ssl_tls.c
@@ -2345,6 +2345,13 @@
         return( POLARSSL_ERR_SSL_BAD_HS_CERTIFICATE );
     }
 
+    /* In case we tried to reuse a session but it failed */
+    if( ssl->session_negotiate->peer_cert != NULL )
+    {
+        x509_free( ssl->session_negotiate->peer_cert );
+        free( ssl->session_negotiate->peer_cert );
+    }
+
     if( ( ssl->session_negotiate->peer_cert = (x509_cert *) malloc(
                     sizeof( x509_cert ) ) ) == NULL )
     {