Silence warning from clang
authorBradley T. Hughes <bradley.hughes@nokia.com>
Tue, 1 Nov 2011 12:53:13 +0000 (13:53 +0100)
committerQt by Nokia <qt-info@nokia.com>
Tue, 1 Nov 2011 16:39:32 +0000 (17:39 +0100)
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 <thiago.macieira@intel.com>
src/3rdparty/md5/md5.cpp

index ac296b5..16871e6 100644 (file)
@@ -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