Skip to content

Commit

Permalink
selftests/bpf: Add a test case to write into .rodata
Browse files Browse the repository at this point in the history
Add a test case which attempts to write into .rodata section of the
BPF program, and for comparison this adds test cases also for .bss
and .data section.

Before fix:

  # ./vmtest.sh -- ./test_progs -t verifier_const
  [...]
  ./test_progs -t verifier_const
  tester_init:PASS:tester_log_buf 0 nsec
  process_subtest:PASS:obj_open_mem 0 nsec
  process_subtest:PASS:specs_alloc 0 nsec
  run_subtest:PASS:obj_open_mem 0 nsec
  run_subtest:FAIL:unexpected_load_success unexpected success: 0
  torvalds#465/1   verifier_const/rodata: write rejected:FAIL
  torvalds#465/2   verifier_const/bss: write accepted:OK
  torvalds#465/3   verifier_const/data: write accepted:OK
  torvalds#465     verifier_const:FAIL
  [...]

After fix:

  # ./vmtest.sh -- ./test_progs -t verifier_const
  [...]
  ./test_progs -t verifier_const
  torvalds#465/1   verifier_const/rodata: write rejected:OK
  torvalds#465/2   verifier_const/bss: write accepted:OK
  torvalds#465/3   verifier_const/data: write accepted:OK
  torvalds#465     verifier_const:OK
  [...]

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Shung-Hsi Yu <shung-hsi.yu@suse.com>
  • Loading branch information
borkmann authored and intel-lab-lkp committed Sep 6, 2024
1 parent 050bba2 commit 8d625aa
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 0 deletions.
2 changes: 2 additions & 0 deletions tools/testing/selftests/bpf/prog_tests/verifier.c
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
#include "verifier_cgroup_inv_retcode.skel.h"
#include "verifier_cgroup_skb.skel.h"
#include "verifier_cgroup_storage.skel.h"
#include "verifier_const.skel.h"
#include "verifier_const_or.skel.h"
#include "verifier_ctx.skel.h"
#include "verifier_ctx_sk_msg.skel.h"
Expand Down Expand Up @@ -146,6 +147,7 @@ void test_verifier_cfg(void) { RUN(verifier_cfg); }
void test_verifier_cgroup_inv_retcode(void) { RUN(verifier_cgroup_inv_retcode); }
void test_verifier_cgroup_skb(void) { RUN(verifier_cgroup_skb); }
void test_verifier_cgroup_storage(void) { RUN(verifier_cgroup_storage); }
void test_verifier_const(void) { RUN(verifier_const); }
void test_verifier_const_or(void) { RUN(verifier_const_or); }
void test_verifier_ctx(void) { RUN(verifier_ctx); }
void test_verifier_ctx_sk_msg(void) { RUN(verifier_ctx_sk_msg); }
Expand Down
42 changes: 42 additions & 0 deletions tools/testing/selftests/bpf/progs/verifier_const.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
// SPDX-License-Identifier: GPL-2.0
/* Copyright (c) 2024 Isovalent */

#include <linux/bpf.h>
#include <bpf/bpf_helpers.h>
#include "bpf_misc.h"

const volatile long foo = 42;
long bar;
long bart = 96;

SEC("tc/ingress")
__description("rodata: write rejected")
__failure __msg("write into map forbidden")
int tcx1(struct __sk_buff *skb)
{
char buff[] = { '8', '4', '\0' };
bpf_strtol(buff, sizeof(buff), 0, (long *)&foo);
return TCX_PASS;
}

SEC("tc/ingress")
__description("bss: write accepted")
__success
int tcx2(struct __sk_buff *skb)
{
char buff[] = { '8', '4', '\0' };
bpf_strtol(buff, sizeof(buff), 0, &bar);
return TCX_PASS;
}

SEC("tc/ingress")
__description("data: write accepted")
__success
int tcx3(struct __sk_buff *skb)
{
char buff[] = { '8', '4', '\0' };
bpf_strtol(buff, sizeof(buff), 0, &bart);
return TCX_PASS;
}

char LICENSE[] SEC("license") = "GPL";

0 comments on commit 8d625aa

Please sign in to comment.