From 9ea966e79a7d698fd2622fa5ec27101c13f7cefc 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 (cherry-picked from qtbase commit 2bce4627e63947a3ed55009aeb8ff38974e444d3) Reviewed-by: Olivier Goffart --- src/3rdparty/md5/md5.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/3rdparty/md5/md5.cpp b/src/3rdparty/md5/md5.cpp index ac296b5..16871e6 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 v0.12