diff options
| author | Volker Lendecke <vl@samba.org> | 2011-03-27 20:03:23 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2011-03-27 22:22:10 +0200 |
| commit | a431394ce37a3f647953969c85aac4415184a532 (patch) | |
| tree | fd3e23e6c0d2718109d768933d67ff2e330f375f | |
| parent | 8db0374251b72c3fc82367864da178dc842f588b (diff) | |
| download | samba-a431394ce37a3f647953969c85aac4415184a532.tar.gz samba-a431394ce37a3f647953969c85aac4415184a532.tar.bz2 samba-a431394ce37a3f647953969c85aac4415184a532.zip | |
s3: Fix Coverity ID 958: BAD_SIZEOF
This is supposed to wipe out the md5 context, not only the first bytes of it.
Others, please check!
| -rw-r--r-- | lib/crypto/md5.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/crypto/md5.c b/lib/crypto/md5.c index e6178cc015..032474478e 100644 --- a/lib/crypto/md5.c +++ b/lib/crypto/md5.c @@ -144,7 +144,7 @@ _PUBLIC_ void MD5Final(uint8_t digest[16], struct MD5Context *ctx) MD5Transform(ctx->buf, (uint32_t *) ctx->in); byteReverse((uint8_t *) ctx->buf, 4); memmove(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 */ |