commit d220f8b709e54e49854c5c75ac51e68d4349fb97
author Manuel Pégourié-Gonnard <mpg@elzevir.fr> Mon Jan 20 10:03:15 2014 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Mon Jul 07 16:01:33 2014 +0200
tree 82c6d67341913f969ab224265024e1aad7a0d9cc
parent 7fd620b331970e5db9f6721470c4f4f57de73fa6

Fix potential memory leak in bignum selftest

library/bignum.c

diff --git a/library/bignum.c b/library/bignum.c
index c26842f..2420866 100644
--- a/library/bignum.c
+++ b/library/bignum.c
@@ -2028,7 +2028,8 @@
         if( verbose != 0 )
             printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -2053,7 +2054,8 @@
         if( verbose != 0 )
             printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -2074,7 +2076,8 @@
         if( verbose != 0 )
             printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -2096,7 +2099,8 @@
         if( verbose != 0 )
             printf( "failed\n" );
 
-        return( 1 );
+        ret = 1;
+        goto cleanup;
     }
 
     if( verbose != 0 )
@@ -2111,15 +2115,16 @@
         MPI_CHK( mpi_lset( &X, gcd_pairs[i][0] ) );
         MPI_CHK( mpi_lset( &Y, gcd_pairs[i][1] ) );
 
-	    MPI_CHK( mpi_gcd( &A, &X, &Y ) );
+        MPI_CHK( mpi_gcd( &A, &X, &Y ) );
 
-	    if( mpi_cmp_int( &A, gcd_pairs[i][2] ) != 0 )
-	    {
-		    if( verbose != 0 )
-			    printf( "failed at %d\n", i );
+        if( mpi_cmp_int( &A, gcd_pairs[i][2] ) != 0 )
+        {
+            if( verbose != 0 )
+                printf( "failed at %d\n", i );
 
-		    return( 1 );
-	    }
+            ret = 1;
+            goto cleanup;
+        }
     }
 
     if( verbose != 0 )