diff options
| author | sf <sf@13f79535-47bb-0310-9956-ffa450edef68> | 2011-09-19 21:11:38 +0000 |
|---|---|---|
| committer | sf <sf@13f79535-47bb-0310-9956-ffa450edef68> | 2011-09-19 21:11:38 +0000 |
| commit | 1b0ff2c0b4787b7f883a0e6a9f0508a5491d05e6 (patch) | |
| tree | 643a7d0a3cda645f13fb41e302331b0f6f51755d /random | |
| parent | 4d6597ed7ff35eec9b543176d4c04cb3a7eee057 (diff) | |
| download | libapr-1b0ff2c0b4787b7f883a0e6a9f0508a5491d05e6.tar.gz | |
Fix issue found by PVS-Studio static analyzer:
Correctly clear whole sha256 state
PR: 51542
Submitted by: Andrey Karpov <karpov viva64 com>
git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@1172825 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'random')
| -rw-r--r-- | random/unix/sha2.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/random/unix/sha2.c b/random/unix/sha2.c index 070526d9d..212c1b732 100644 --- a/random/unix/sha2.c +++ b/random/unix/sha2.c @@ -557,7 +557,7 @@ void apr__SHA256_Final(sha2_byte digest[], SHA256_CTX* context) { } /* Clean up state data: */ - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); usedspace = 0; } @@ -578,7 +578,7 @@ char *apr__SHA256_End(SHA256_CTX* context, char buffer[]) { } *buffer = (char)0; } else { - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); } MEMSET_BZERO(digest, SHA256_DIGEST_LENGTH); return buffer; @@ -889,7 +889,7 @@ void apr__SHA512_Final(sha2_byte digest[], SHA512_CTX* context) { } /* Zero out state data */ - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); } char *apr__SHA512_End(SHA512_CTX* context, char buffer[]) { @@ -909,7 +909,7 @@ char *apr__SHA512_End(SHA512_CTX* context, char buffer[]) { } *buffer = (char)0; } else { - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); } MEMSET_BZERO(digest, SHA512_DIGEST_LENGTH); return buffer; @@ -964,7 +964,7 @@ void apr__SHA384_Final(sha2_byte digest[], SHA384_CTX* context) { } /* Zero out state data */ - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); } char *apr__SHA384_End(SHA384_CTX* context, char buffer[]) { @@ -984,7 +984,7 @@ char *apr__SHA384_End(SHA384_CTX* context, char buffer[]) { } *buffer = (char)0; } else { - MEMSET_BZERO(context, sizeof(context)); + MEMSET_BZERO(context, sizeof(*context)); } MEMSET_BZERO(digest, SHA384_DIGEST_LENGTH); return buffer; |