diff options
| author | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-02-14 18:35:35 +0100 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-02-14 18:35:35 +0100 |
| commit | 442ae26d495655e4a16da9201542ae37f7140fb3 (patch) | |
| tree | d401e7c71bcc003a35caf1edce3d8af8d7a9cfe1 /src/crypto/blake2s.c | |
| parent | db0e44139b6db098a6636eb5e6b74390af45dd3f (diff) | |
blake2s: use union instead of casting
This deals with alignment more easily and also helps squelch a
clang-analyzer warning.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'src/crypto/blake2s.c')
| -rw-r--r-- | src/crypto/blake2s.c | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/src/crypto/blake2s.c b/src/crypto/blake2s.c index 2fbaf09..1bb3cc1 100644 --- a/src/crypto/blake2s.c +++ b/src/crypto/blake2s.c @@ -13,18 +13,21 @@ #include <linux/bug.h> #include <asm/unaligned.h> -typedef struct { - u8 digest_length; - u8 key_length; - u8 fanout; - u8 depth; - u32 leaf_length; - u32 node_offset; - u16 xof_length; - u8 node_depth; - u8 inner_length; - u8 salt[8]; - u8 personal[8]; +typedef union { + struct { + u8 digest_length; + u8 key_length; + u8 fanout; + u8 depth; + u32 leaf_length; + u32 node_offset; + u16 xof_length; + u8 node_depth; + u8 inner_length; + u8 salt[8]; + u8 personal[8]; + }; + __le32 words[8]; } __packed blake2s_param; static const u32 blake2s_iv[8] = { @@ -65,16 +68,11 @@ static inline void blake2s_increment_counter(struct blake2s_state *state, const static inline void blake2s_init_param(struct blake2s_state *state, const blake2s_param *param) { - const __le32 *p; int i; memset(state, 0, sizeof(struct blake2s_state)); for (i = 0; i < 8; ++i) - state->h[i] = blake2s_iv[i]; - p = (const __le32 *)param; - /* IV XOR ParamBlock */ - for (i = 0; i < 8; ++i) - state->h[i] ^= le32_to_cpu(p[i]); + state->h[i] = blake2s_iv[i] ^ le32_to_cpu(param->words[i]); } void blake2s_init(struct blake2s_state *state, const size_t outlen) |