commit 346177255931fb1e3514e124a1f598715a5fe92d
author Paul Bakker <p.j.bakker@polarssl.org> Fri Jun 13 17:20:13 2014 +0200
committer Paul Bakker <p.j.bakker@polarssl.org> Sat Jun 14 16:46:03 2014 +0200
tree da658994944be861b8b2a1001c9b759976276891
parent bbcb1ce7030575d708700ca205800e847d846402

Introduce polarssl_zeroize() instead of memset() for zeroization

library/ripemd160.c

diff --git a/library/ripemd160.c b/library/ripemd160.c
index 4781a7d..3c2a7e6 100644
--- a/library/ripemd160.c
+++ b/library/ripemd160.c
@@ -76,6 +76,11 @@
 }
 #endif
 
+/* Implementation that should never be optimized out by the compiler */
+static void polarssl_zeroize( void *v, size_t n ) {
+    volatile unsigned char *p = v; while( n-- ) *p++ = 0;
+}
+
 /*
  * RIPEMD-160 context setup
  */
@@ -363,7 +368,7 @@
     ripemd160_update( &ctx, input, ilen );
     ripemd160_finish( &ctx, output );
 
-    memset( &ctx, 0, sizeof( ripemd160_context ) );
+    polarssl_zeroize( &ctx, sizeof( ripemd160_context ) );
 }
 
 #if defined(POLARSSL_FS_IO)
@@ -387,7 +392,7 @@
 
     ripemd160_finish( &ctx, output );
 
-    memset( &ctx, 0, sizeof( ripemd160_context ) );
+    polarssl_zeroize( &ctx, sizeof( ripemd160_context ) );
 
     if( ferror( f ) != 0 )
     {
@@ -428,7 +433,7 @@
     ripemd160_starts( ctx );
     ripemd160_update( ctx, ctx->ipad, 64 );
 
-    memset( sum, 0, sizeof( sum ) );
+    polarssl_zeroize( sum, sizeof( sum ) );
 }
 
 /*
@@ -453,7 +458,7 @@
     ripemd160_update( ctx, tmpbuf, 20 );
     ripemd160_finish( ctx, output );
 
-    memset( tmpbuf, 0, sizeof( tmpbuf ) );
+    polarssl_zeroize( tmpbuf, sizeof( tmpbuf ) );
 }
 
 /*
@@ -478,7 +483,7 @@
     ripemd160_hmac_update( &ctx, input, ilen );
     ripemd160_hmac_finish( &ctx, output );
 
-    memset( &ctx, 0, sizeof( ripemd160_context ) );
+    polarssl_zeroize( &ctx, sizeof( ripemd160_context ) );
 }