diff options
| author | Sidong Yang <sidong.yang@furiosa.ai> | 2024-11-16 17:10:52 +0900 |
|---|---|---|
| committer | Alexei Starovoitov <ast@kernel.org> | 2024-11-16 11:01:38 -0800 |
| commit | 2c8b09ac2537299511c898bc71b1a5f2756c831c (patch) | |
| tree | 4cb993bba29d1c359464be75aec36cc9af15a04e /tools/lib | |
| parent | 608e99f7869e3a6e028c7cba14a896c7797e8746 (diff) | |
libbpf: Change hash_combine parameters from long to unsigned long
The hash_combine() could be trapped when compiled with sanitizer like "zig cc"
or clang with signed-integer-overflow option. This patch parameters and return
type to unsigned long to remove the potential overflow.
Signed-off-by: Sidong Yang <sidong.yang@furiosa.ai>
Acked-by: Yonghong Song <yonghong.song@linux.dev>
Link: https://lore.kernel.org/r/20241116081054.65195-1-sidong.yang@furiosa.ai
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/lib')
| -rw-r--r-- | tools/lib/bpf/btf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/lib/bpf/btf.c b/tools/lib/bpf/btf.c index 8befb8103e32..12468ae0d573 100644 --- a/tools/lib/bpf/btf.c +++ b/tools/lib/bpf/btf.c @@ -3548,7 +3548,7 @@ struct btf_dedup { struct strset *strs_set; }; -static long hash_combine(long h, long value) +static unsigned long hash_combine(unsigned long h, unsigned long value) { return h * 31 + value; } |