Skip to content

Commit 192e3aa

Browse files
pchaignoAlexei Starovoitov
authored andcommitted
selftests/bpf: Negative test case for tail call map
This patch adds a negative test case for the following verifier error. expected prog array map for tail call Acked-by: Yonghong Song <[email protected]> Signed-off-by: Paul Chaignon <[email protected]> Acked-by: Daniel Borkmann <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Alexei Starovoitov <[email protected]>
1 parent df4b1ee commit 192e3aa

File tree

2 files changed

+33
-0
lines changed

2 files changed

+33
-0
lines changed

tools/testing/selftests/bpf/prog_tests/verifier.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@
8585
#include "verifier_store_release.skel.h"
8686
#include "verifier_subprog_precision.skel.h"
8787
#include "verifier_subreg.skel.h"
88+
#include "verifier_tailcall.skel.h"
8889
#include "verifier_tailcall_jit.skel.h"
8990
#include "verifier_typedef.skel.h"
9091
#include "verifier_uninit.skel.h"
@@ -219,6 +220,7 @@ void test_verifier_stack_ptr(void) { RUN(verifier_stack_ptr); }
219220
void test_verifier_store_release(void) { RUN(verifier_store_release); }
220221
void test_verifier_subprog_precision(void) { RUN(verifier_subprog_precision); }
221222
void test_verifier_subreg(void) { RUN(verifier_subreg); }
223+
void test_verifier_tailcall(void) { RUN(verifier_tailcall); }
222224
void test_verifier_tailcall_jit(void) { RUN(verifier_tailcall_jit); }
223225
void test_verifier_typedef(void) { RUN(verifier_typedef); }
224226
void test_verifier_uninit(void) { RUN(verifier_uninit); }
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
// SPDX-License-Identifier: GPL-2.0
2+
3+
#include <linux/bpf.h>
4+
#include <bpf/bpf_helpers.h>
5+
#include "bpf_misc.h"
6+
7+
struct {
8+
__uint(type, BPF_MAP_TYPE_ARRAY);
9+
__uint(max_entries, 1);
10+
__type(key, __u32);
11+
__type(value, __u32);
12+
} map_array SEC(".maps");
13+
14+
SEC("socket")
15+
__description("invalid map type for tail call")
16+
__failure __msg("expected prog array map for tail call")
17+
__failure_unpriv
18+
__naked void invalid_map_for_tail_call(void)
19+
{
20+
asm volatile (" \
21+
r2 = %[map_array] ll; \
22+
r3 = 0; \
23+
call %[bpf_tail_call]; \
24+
exit; \
25+
" :
26+
: __imm(bpf_tail_call),
27+
__imm_addr(map_array)
28+
: __clobber_all);
29+
}
30+
31+
char _license[] SEC("license") = "GPL";

0 commit comments

Comments
 (0)