diff options
| author | Jouni Malinen <j@w1.fi> | 2011-11-13 23:24:08 +0200 |
|---|---|---|
| committer | Jouni Malinen <j@w1.fi> | 2011-11-13 23:24:08 +0200 |
| commit | 8f2e2e369b4ebc3bc609f1eba865300374397b3f (patch) | |
| tree | ac0b6492367ebef55ced0a9d79fd62c142066f68 /src/crypto | |
| parent | 911e10eac6563b86ef5f8d2ecb82574c29f8e4ae (diff) | |
| download | external_wpa_supplicant_8_ti-8f2e2e369b4ebc3bc609f1eba865300374397b3f.zip external_wpa_supplicant_8_ti-8f2e2e369b4ebc3bc609f1eba865300374397b3f.tar.gz external_wpa_supplicant_8_ti-8f2e2e369b4ebc3bc609f1eba865300374397b3f.tar.bz2 | |
Make crypto_hash_init() easier for static analyzers
Avoid zero-length memset at the end of the buffer. This is not really
needed, but it makes the code a bit easier for static analyzers.
Signed-hostap: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'src/crypto')
| -rw-r--r-- | src/crypto/crypto_internal.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/crypto/crypto_internal.c b/src/crypto/crypto_internal.c index 955cd55..5f715a8 100644 --- a/src/crypto/crypto_internal.c +++ b/src/crypto/crypto_internal.c @@ -82,7 +82,8 @@ struct crypto_hash * crypto_hash_init(enum crypto_hash_alg alg, const u8 *key, ctx->key_len = key_len; os_memcpy(k_pad, key, key_len); - os_memset(k_pad + key_len, 0, sizeof(k_pad) - key_len); + if (key_len < sizeof(k_pad)) + os_memset(k_pad + key_len, 0, sizeof(k_pad) - key_len); for (i = 0; i < sizeof(k_pad); i++) k_pad[i] ^= 0x36; SHA1Init(&ctx->u.sha1); |