diff options
| author | Daniel Borkmann <[email protected]> | 2021-01-19 23:24:24 +0000 |
|---|---|---|
| committer | Daniel Borkmann <[email protected]> | 2021-01-20 16:19:40 +0000 |
| commit | bc895e8b2a64e502fbba72748d59618272052a8b (patch) | |
| tree | 2f161e03d816093c3ee642a48cd920ab2a3288bf /tools/perf/builtin-script.c | |
| parent | xsk: Clear pool even for inactive queues (diff) | |
| download | kernel-bc895e8b2a64e502fbba72748d59618272052a8b.tar.gz kernel-bc895e8b2a64e502fbba72748d59618272052a8b.zip | |
bpf: Fix signed_{sub,add32}_overflows type handling
Fix incorrect signed_{sub,add32}_overflows() input types (and a related buggy
comment). It looks like this might have slipped in via copy/paste issue, also
given prior to 3f50f132d840 ("bpf: Verifier, do explicit ALU32 bounds tracking")
the signature of signed_sub_overflows() had s64 a and s64 b as its input args
whereas now they are truncated to s32. Thus restore proper types. Also, the case
of signed_add32_overflows() is not consistent to signed_sub32_overflows(). Both
have s32 as inputs, therefore align the former.
Fixes: 3f50f132d840 ("bpf: Verifier, do explicit ALU32 bounds tracking")
Reported-by: De4dCr0w <[email protected]>
Signed-off-by: Daniel Borkmann <[email protected]>
Reviewed-by: John Fastabend <[email protected]>
Acked-by: Alexei Starovoitov <[email protected]>
Diffstat (limited to 'tools/perf/builtin-script.c')
0 files changed, 0 insertions, 0 deletions