summaryrefslogtreecommitdiff
path: root/tools/spi
diff options
context:
space:
mode:
authorVishwanath Pai <vpai@akamai.com>2017-09-08 01:38:58 -0400
committerPablo Neira Ayuso <pablo@netfilter.org>2017-09-08 18:55:53 +0200
commit90c4ae4e2c1da9f1eaf846136861af43d4c1ff34 (patch)
tree3d78647f4cf96a1ef30a8169d461cf0a46a86899 /tools/spi
parent05d0eae7c1cf6bf7b19c02b3a97ff457b3317323 (diff)
netfilter: xt_hashlimit: fix build error caused by 64bit division
64bit division causes build/link errors on 32bit architectures. It prints out error messages like: ERROR: "__aeabi_uldivmod" [net/netfilter/xt_hashlimit.ko] undefined! The value of avg passed through by userspace in BYTE mode cannot exceed U32_MAX. Which means 64bit division in user2rate_bytes is unnecessary. To fix this I have changed the type of param 'user' to u32. Since anything greater than U32_MAX is an invalid input we error out in hashlimit_mt_check_common() when this is the case. Changes in v2: Making return type as u32 would cause an overflow for small values of 'user' (for example 2, 3 etc). To avoid this I bumped up 'r' to u64 again as well as the return type. This is OK since the variable that stores the result is u64. We still avoid 64bit division here since 'user' is u32. Fixes: bea74641e378 ("netfilter: xt_hashlimit: add rate match mode") Signed-off-by: Vishwanath Pai <vpai@akamai.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'tools/spi')
0 files changed, 0 insertions, 0 deletions