Subtle memset bug in MD5
authorChip Black <bytex64@bytex64.net>
Sat, 22 Mar 2014 22:12:04 +0000 (17:12 -0500)
committerChip Black <bytex64@bytex64.net>
Sat, 29 Mar 2014 03:56:25 +0000 (22:56 -0500)
common/md5.c

index f19672e..ffbc215 100644 (file)
@@ -257,7 +257,7 @@ void MD5Final(unsigned char digest[16],
     MD5Transform(ctx->buf, (uint32_t *) ctx->in);
     byteReverse((unsigned char *) ctx->buf, 4);
     memcpy(digest, ctx->buf, 16);
-    memset(ctx, 0, sizeof(ctx));        /* In case it's sensitive */
+    memset(ctx, 0, sizeof(struct MD5Context));        /* In case it's sensitive */
 }
 
 /* end of md5.c */