commit 02303e8be478dc8836093331bde1341936ce1dc9
author Paul Bakker <p.j.bakker@polarssl.org> Thu Jan 03 11:08:31 2013 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Thu Jan 03 11:08:31 2013 +0100
tree 97f3e03fc1d76e6d39e9b0d81b9daa5c5f800f70
parent 40628bad98973fb7270b6822924086c4d27b3b79

Moved md_init_ctx() calls around to minimize exit points

library/rsa.c

diff --git a/library/rsa.c b/library/rsa.c
index 5d7129a..ee6ca01 100644
--- a/library/rsa.c
+++ b/library/rsa.c
@@ -441,8 +441,6 @@
 
             memset( output, 0, olen );
 
-            md_init_ctx( &md_ctx, md_info );
-
             *p++ = 0;
 
             // Generate a random octet string seed
@@ -460,6 +458,8 @@
             *p++ = 1;
             memcpy( p, input, ilen ); 
 
+            md_init_ctx( &md_ctx, md_info );
+
             // maskedDB: Apply dbMask to DB
             //
             mgf_mask( output + hlen + 1, olen - hlen - 1, output + 1, hlen,  
@@ -800,8 +800,6 @@
 
             memset( sig, 0, olen );
 
-            md_init_ctx( &md_ctx, md_info );
-
             msb = mpi_msb( &ctx->N ) - 1;
 
             // Generate salt of length slen
@@ -817,6 +815,8 @@
             memcpy( p, salt, slen );
             p += slen;
 
+            md_init_ctx( &md_ctx, md_info );
+
             // Generate H = Hash( M' )
             //
             md_starts( &md_ctx );
@@ -1016,8 +1016,6 @@
 
             memset( zeros, 0, 8 );
 
-            md_init_ctx( &md_ctx, md_info );
-
             // Note: EMSA-PSS verification is over the length of N - 1 bits
             //
             msb = mpi_msb( &ctx->N ) - 1;
@@ -1032,6 +1030,8 @@
             if( buf[0] >> ( 8 - siglen * 8 + msb ) )
                 return( POLARSSL_ERR_RSA_BAD_INPUT_DATA );
 
+            md_init_ctx( &md_ctx, md_info );
+
             mgf_mask( p, siglen - hlen - 1, p + siglen - hlen - 1, hlen, &md_ctx );
 
             buf[0] &= 0xFF >> ( siglen * 8 - msb );
@@ -1039,11 +1039,12 @@
             while( *p == 0 && p < buf + siglen )
                 p++;
 
-            if( p == buf + siglen )
+            if( p == buf + siglen ||
+                *p++ != 0x01 )
+            {
+                md_free_ctx( &md_ctx );
                 return( POLARSSL_ERR_RSA_INVALID_PADDING );
-
-            if( *p++ != 0x01 )
-                return( POLARSSL_ERR_RSA_INVALID_PADDING );
+            }
 
             slen -= p - buf;