selftests/bpf: Fix segmentation fault in test_progs
authorJianlin Lv <Jianlin.Lv@arm.com>
Mon, 10 Aug 2020 15:39:40 +0000 (23:39 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Tue, 11 Aug 2020 13:36:45 +0000 (15:36 +0200)
commit0390c429dbed4068bd2cd8dded937d9a5ec24cd2
tree467e1b67bae48778d9b916e6e42e0a7f4d3b62cc
parent63fe3fd393dc4e7ea3948e79947362ffbb0fd616
selftests/bpf: Fix segmentation fault in test_progs

test_progs reports the segmentation fault as below:

  $ sudo ./test_progs -t mmap --verbose
  test_mmap:PASS:skel_open_and_load 0 nsec
  [...]
  test_mmap:PASS:adv_mmap1 0 nsec
  test_mmap:PASS:adv_mmap2 0 nsec
  test_mmap:PASS:adv_mmap3 0 nsec
  test_mmap:PASS:adv_mmap4 0 nsec
  Segmentation fault

This issue was triggered because mmap() and munmap() used inconsistent
length parameters; mmap() creates a new mapping of 3 * page_size, but the
length parameter set in the subsequent re-map and munmap() functions is
4 * page_size; this leads to the destruction of the process space.

To fix this issue, first create 4 pages of anonymous mapping, then do all
the mmap() with MAP_FIXED.

Another issue is that when unmap the second page fails, the length
parameter to delete tmp1 mappings should be 4 * page_size.

Signed-off-by: Jianlin Lv <Jianlin.Lv@arm.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Link: https://lore.kernel.org/bpf/20200810153940.125508-1-Jianlin.Lv@arm.com
tools/testing/selftests/bpf/prog_tests/mmap.c