From 2bce4627e63947a3ed55009aeb8ff38974e444d3 Mon Sep 17 00:00:00 2001 From: "Bradley T. Hughes" Date: Tue, 1 Nov 2011 13:53:13 +0100 Subject: Silence warning from clang The original intent was most likely to memset() the entire MD5Context to zero at the end of MD5Final(), which we do now. In file included from tools/qcryptographichash.cpp:49: ...qtbase/src/corelib/../../include/QtCore/../../src/corelib/tools/../../3rdparty/md5/md5.cpp:139:24: warning: argument to 'sizeof' in 'memset' call is the same expression as the destination; did you mean to dereference it? [-Wsizeof-pointer- memset(ctx, 0, sizeof(ctx)); /* In case it's sensitive */ ~~~ ^~~ Change-Id: I793c6f0944b89c0e4c5f9253cdb1071175c17152 Reviewed-by: Thiago Macieira --- src/3rdparty/md5/md5.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/3rdparty/md5') diff --git a/src/3rdparty/md5/md5.cpp b/src/3rdparty/md5/md5.cpp index ac296b532b..16871e6626 100644 --- a/src/3rdparty/md5/md5.cpp +++ b/src/3rdparty/md5/md5.cpp @@ -136,7 +136,7 @@ MD5Final(struct MD5Context *ctx, md5byte digest[16]) byteSwap(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 */ } #ifndef ASM_MD5 -- cgit v1.2.3