From 3c59c247e28a63514a186781d68cf32a4a32e62f Mon Sep 17 00:00:00 2001 From: "mark@chromium.org" Date: Thu, 11 Aug 2011 13:39:44 +0000 Subject: Zero out the entire context on MD5_Final, not just the number of bytes in a pointer. Review URL: http://breakpad.appspot.com/298001 git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@822 4c0a9323-5329-0410-9bdc-e9ce6186880e --- src/common/md5.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/common/md5.c b/src/common/md5.c index 7fc198af..539bd6d2 100644 --- a/src/common/md5.c +++ b/src/common/md5.c @@ -142,7 +142,7 @@ void MD5Final(unsigned char digest[16], struct MD5Context *ctx) MD5Transform(ctx->buf, (u32 *) 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(*ctx)); /* In case it's sensitive */ } /* The four core functions - F1 is optimized somewhat */ -- cgit v1.2.1