diff options
| author |   Johan Almbladh <johan.almbladh@anyfinetworks.com> | 2021-07-28 18:47:41 +0200 |
|---|---|---|
| committer |   Andrii Nakryiko <andrii@kernel.org> | 2021-08-02 15:05:43 -0700 |
| commit | b61a28cf11d61f512172e673b8f8c4a6c789b425 (patch) | |
| tree | 65723b716120462a8be617eda038846a59045dca /kernel/bpf | |
| parent | net/mlx4: make the array states static const, makes object smaller (diff) | |
| download | linux-dev-b61a28cf11d61f512172e673b8f8c4a6c789b425.tar.xz linux-dev-b61a28cf11d61f512172e673b8f8c4a6c789b425.zip | |
bpf: Fix off-by-one in tail call count limiting
Before, the interpreter allowed up to MAX_TAIL_CALL_CNT + 1 tail calls.
Now precisely MAX_TAIL_CALL_CNT is allowed, which is in line with the
behavior of the x86 JITs.
Signed-off-by: Johan Almbladh <johan.almbladh@anyfinetworks.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Acked-by: Yonghong Song <yhs@fb.com>
Link: https://lore.kernel.org/bpf/20210728164741.350370-1-johan.almbladh@anyfinetworks.com
Diffstat (limited to 'kernel/bpf')
| -rw-r--r-- | kernel/bpf/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/bpf/core.c b/kernel/bpf/core.c index b1a5fc04492b..fe807b203a6f 100644 --- a/kernel/bpf/core.c +++ b/kernel/bpf/core.c @@ -1562,7 +1562,7 @@ select_insn: if (unlikely(index >= array->map.max_entries)) goto out; - if (unlikely(tail_call_cnt > MAX_TAIL_CALL_CNT)) + if (unlikely(tail_call_cnt >= MAX_TAIL_CALL_CNT)) goto out; tail_call_cnt++; |