diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-01-04 12:30:45 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-01-04 12:37:25 +0100 |
commit | 3042bbebddb819371aa2ee0811a1f9d3d38f7b94 (patch) | |
tree | 50a6d8b8a03ef12c9842fe53cbc9577b04fed09e /src/basic/siphash24.c | |
parent | d27d60b3bca61684a2b6e14f984e02d8a7ecd15c (diff) | |
download | systemd-3042bbebddb819371aa2ee0811a1f9d3d38f7b94.tar.gz |
tree-wide: use c99 static for array size declarations
https://hamberg.no/erlend/posts/2013-02-18-static-array-indices.html
This only works with clang, unfortunately gcc doesn't seem to implement the check
(tested with gcc-8.2.1-5.fc29.x86_64).
Simulated error:
[2/3] Compiling C object 'systemd-nspawn@exe/src_nspawn_nspawn.c.o'.
../src/nspawn/nspawn.c:3179:45: warning: array argument is too small; contains 15 elements, callee requires at least 16 [-Warray-bounds]
candidate = (uid_t) siphash24(arg_machine, strlen(arg_machine), hash_key);
^ ~~~~~~~~
../src/basic/siphash24.h:24:64: note: callee declares array parameter as static here
uint64_t siphash24(const void *in, size_t inlen, const uint8_t k[static 16]);
^~~~~~~~~~~~
Diffstat (limited to 'src/basic/siphash24.c')
-rw-r--r-- | src/basic/siphash24.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/basic/siphash24.c b/src/basic/siphash24.c index 30c228a78a..bf365b505d 100644 --- a/src/basic/siphash24.c +++ b/src/basic/siphash24.c @@ -48,7 +48,7 @@ static inline void sipround(struct siphash *state) { state->v2 = rotate_left(state->v2, 32); } -void siphash24_init(struct siphash *state, const uint8_t k[16]) { +void siphash24_init(struct siphash *state, const uint8_t k[static 16]) { uint64_t k0, k1; assert(state); @@ -187,7 +187,7 @@ uint64_t siphash24_finalize(struct siphash *state) { return state->v0 ^ state->v1 ^ state->v2 ^ state->v3; } -uint64_t siphash24(const void *in, size_t inlen, const uint8_t k[16]) { +uint64_t siphash24(const void *in, size_t inlen, const uint8_t k[static 16]) { struct siphash state; assert(in); |