diff options
| author | Daniel Borkmann <daniel@iogearbox.net> | 2026-05-29 12:41:16 +0300 |
|---|---|---|
| committer | Alexei Starovoitov <ast@kernel.org> | 2026-06-01 03:47:48 +0300 |
| commit | e2c88266147ff92ca25e6577158a9a0b3b261a30 (patch) | |
| tree | 8939a4eb0f19c5c8272b25657ffd2899dc94e553 /include/linux/timerqueue.h | |
| parent | 7c7c42d606ed540301b14571ae000041a2d6f39d (diff) | |
| download | linux-e2c88266147ff92ca25e6577158a9a0b3b261a30.tar.xz | |
libbpf: Drop redundant self-loop in emit_check_err
When the cleanup-label jump offset does not fit in s16, emit_check_err()
sets gen->error = -ERANGE and then emits a BPF_JMP_IMM(BPF_JA, 0, 0, -1)
self-loop.
The latter emit() is dead: gen->error is assigned on the preceding line,
and emit() then bails out early in realloc_insn_buf() the moment gen->error
is set, so the jump is never written into the instruction stream.
gen->error alone already marks the generation as failed. This is a follow-up
to 7dd62566e0d1 ("libbpf: fix off-by-one in emit_signature_match jump offset")
which removed the jump in emit_signature_match() but not in other locations.
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/r/20260529094119.307264-1-daniel@iogearbox.net
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'include/linux/timerqueue.h')
0 files changed, 0 insertions, 0 deletions
