commit 79f1ff84edc219a6b3506df93763583086322e5e
author Manuel Pégourié-Gonnard <mpg@elzevir.fr> Mon Oct 28 12:51:32 2013 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Mon Jul 07 14:07:23 2014 +0200
tree ae97cc48fcf30d743bd7430f12091b574da9e757
parent 2a8c2881f4158b018011eaa77ab8cfb8761389a5

Make all hash checking in programs constant-time

programs/hash/md5sum.c

diff --git a/programs/hash/md5sum.c b/programs/hash/md5sum.c
index 1ca7e87..ede3c45 100644
--- a/programs/hash/md5sum.c
+++ b/programs/hash/md5sum.c
@@ -81,6 +81,7 @@
     int nb_tot1, nb_tot2;
     unsigned char sum[16];
     char buf[33], line[1024];
+    char diff;
 
     if( ( f = fopen( filename, "rb" ) ) == NULL )
     {
@@ -121,7 +122,12 @@
         for( i = 0; i < 16; i++ )
             sprintf( buf + i * 2, "%02x", sum[i] );
 
-        if( memcmp( line, buf, 32 ) != 0 )
+        /* Use constant-time buffer comparison */
+        diff = 0;
+        for( i = 0; i < 32; i++ )
+            diff |= line[i] ^ buf[i];
+
+        if( diff != 0 )
         {
             nb_err2++;
             fprintf( stderr, "wrong checksum: %s\n", line + 34 );