commit 5bc8729b9e7738d8f9a32e96b8e1fb2f597e3609
author Hanno Becker <hanno.becker@arm.com> Wed May 03 15:09:31 2017 +0100
committer Hanno Becker <hanno.becker@arm.com> Fri Jun 09 13:29:53 2017 +0100
tree bc588d189eb22fdd2cb7ce27fe93d50608571c24
parent 88ec2381d6af23935c491272375cea85d942d894

Correct memory leak in RSA self test

The RSA self test didn't free the RSA context on failure.

library/rsa.c

diff --git a/library/rsa.c b/library/rsa.c
index 122bc13..c809004 100644
--- a/library/rsa.c
+++ b/library/rsa.c
@@ -1772,7 +1772,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -1786,7 +1787,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -1799,7 +1801,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( memcmp( rsa_decrypted, rsa_plaintext, len ) != 0 )
@@ -1807,7 +1810,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -1825,7 +1829,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -1837,7 +1842,8 @@
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )