diff options
| author | Bernd Edlinger <bernd.edlinger@hotmail.de> | 2017-05-11 16:21:37 +0200 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2017-05-11 19:39:38 +0200 |
| commit | 018fcbec38509cd03fb0709904a382c3bfcf5ed4 (patch) | |
| tree | cb03c315ac98d80e53a0662f26d69ca9d5ceafbf /crypto/siphash | |
| parent | Unclash clashing reason codes in ssl.h (diff) | |
| download | openssl-018fcbec38509cd03fb0709904a382c3bfcf5ed4.tar.xz openssl-018fcbec38509cd03fb0709904a382c3bfcf5ed4.zip | |
Fix gcc-7 warnings.
- Mostly missing fall thru comments
- And uninitialized value used in sslapitest.c
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3440)
Diffstat (limited to 'crypto/siphash')
| -rw-r--r-- | crypto/siphash/siphash.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/crypto/siphash/siphash.c b/crypto/siphash/siphash.c index 4bf2382972..b00d5889b8 100644 --- a/crypto/siphash/siphash.c +++ b/crypto/siphash/siphash.c @@ -191,16 +191,22 @@ int SipHash_Final(SIPHASH *ctx, unsigned char *out, size_t outlen) switch (ctx->len) { case 7: b |= ((uint64_t)ctx->leavings[6]) << 48; + /* fall thru */ case 6: b |= ((uint64_t)ctx->leavings[5]) << 40; + /* fall thru */ case 5: b |= ((uint64_t)ctx->leavings[4]) << 32; + /* fall thru */ case 4: b |= ((uint64_t)ctx->leavings[3]) << 24; + /* fall thru */ case 3: b |= ((uint64_t)ctx->leavings[2]) << 16; + /* fall thru */ case 2: b |= ((uint64_t)ctx->leavings[1]) << 8; + /* fall thru */ case 1: b |= ((uint64_t)ctx->leavings[0]); case 0: |