diff options
author | Stefano Brivio <sbrivio@redhat.com> | 2021-10-20 00:05:11 +0200 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2021-10-20 08:34:22 +0200 |
commit | 12cfa6444cd239dbc04391027ad3161f53b6901c (patch) | |
tree | 3fa746cef0cf3f543ebbe2e0d1535a331a1f2bd0 /icmp.c | |
parent | 7f1e7019cbd47a13dde38782d4bc7430cecff8c4 (diff) | |
download | passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar.gz passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar.bz2 passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar.lz passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar.xz passt-12cfa6444cd239dbc04391027ad3161f53b6901c.tar.zst passt-12cfa6444cd239dbc04391027ad3161f53b6901c.zip |
passt: Add clang-tidy Makefile target and test, take care of warnings
Most are just about style and form, but a few were actually
serious mistakes (NDP-related).
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'icmp.c')
-rw-r--r-- | icmp.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -254,11 +254,11 @@ void icmp_timer(struct ctx *c, struct timespec *ts) v6: word = (long *)icmp_act[v6 ? V6 : V4]; - for (i = 0; i < sizeof(icmp_act[0]) / sizeof(long); i++, word++) { + for (i = 0; i < ARRAY_SIZE(icmp_act); i += sizeof(long), word++) { tmp = *word; while ((n = ffsl(tmp))) { tmp &= ~(1UL << (n - 1)); - icmp_timer_one(c, v6, i * sizeof(long) * 8 + n - 1, ts); + icmp_timer_one(c, v6, i * 8 + n - 1, ts); } } |