netdev - Re: [PATCH net-next 1/2] bpf: Fix test_bpf_obj_id() when the bpf_jit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 9 Jun 2017 08:39:24 -0700
From:   Yonghong Song <yhs@...com>
To:     Martin KaFai Lau <kafai@...com>, <netdev@...r.kernel.org>
CC:     Alexei Starovoitov <ast@...com>,
        Daniel Borkmann <daniel@...earbox.net>, <kernel-team@...com>
Subject: Re: [PATCH net-next 1/2] bpf: Fix test_bpf_obj_id() when the
 bpf_jit_enable sysctl is diabled



On 6/8/17 10:30 PM, Martin KaFai Lau wrote:
> test_bpf_obj_id() should not expect a non zero jited_prog_len
> to be returned by bpf_obj_get_info_by_fd() when
> net.core.bpf_jit_enable is 0.
> 
> The patch checks for net.core.bpf_jit_enable and
> has different expectation on jited_prog_len.
> 
> This patch also removes the pwd.h header which I forgot
> to remove after making changes.
> 
> Fixes: 95b9afd3987f ("bpf: Test for bpf ID")
> Reported-by: Yonghong Song <yhs@...com>
> Signed-off-by: Martin KaFai Lau <kafai@...com>
> ---
>   tools/testing/selftests/bpf/test_progs.c | 19 +++++++++++++++----
>   1 file changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/testing/selftests/bpf/test_progs.c b/tools/testing/selftests/bpf/test_progs.c
> index 8189bfc7e277..fec13ab84fca 100644
> --- a/tools/testing/selftests/bpf/test_progs.c
> +++ b/tools/testing/selftests/bpf/test_progs.c
> @@ -23,7 +23,7 @@ typedef __u16 __sum16;
>   #include <sys/wait.h>
>   #include <sys/resource.h>
>   #include <sys/types.h>
> -#include <pwd.h>
> +#include <fcntl.h>
>   
>   #include <linux/bpf.h>
>   #include <linux/err.h>
> @@ -297,6 +297,7 @@ static void test_bpf_obj_id(void)
>   	const __u32 array_key = 0;
>   	const int nr_iters = 2;
>   	const char *file = "./test_obj_id.o";
> +	const char *jit_sysctl = "/proc/sys/net/core/bpf_jit_enable";
>   
>   	struct bpf_object *objs[nr_iters];
>   	int prog_fds[nr_iters], map_fds[nr_iters];
> @@ -305,9 +306,18 @@ static void test_bpf_obj_id(void)
>   	struct bpf_map_info map_infos[nr_iters + 1];
>   	char jited_insns[128], xlated_insns[128];
>   	__u32 i, next_id, info_len, nr_id_found, duration = 0;
> -	int err = 0;
> +	int sysctl_fd, jit_enabled = 0, err = 0;
>   	__u64 array_value;
>   
> +	sysctl_fd = open(jit_sysctl, 0, O_RDONLY);
> +	if (sysctl_fd != -1) {
> +		char tmpc;
> +
> +		if (read(sysctl_fd, &tmpc, sizeof(tmpc)) == 1)
> +			jit_enabled = (tmpc != '0');
> +		close(sysctl_fd);
> +	}
> +
>   	err = bpf_prog_get_fd_by_id(0);
>   	CHECK(err >= 0 || errno != ENOENT,
>   	      "get-fd-by-notexist-prog-id", "err %d errno %d\n", err, errno);
> @@ -339,13 +349,14 @@ static void test_bpf_obj_id(void)
>   		if (CHECK(err ||
>   			  prog_infos[i].type != BPF_PROG_TYPE_SOCKET_FILTER ||
>   			  info_len != sizeof(struct bpf_prog_info) ||
> -			  !prog_infos[i].jited_prog_len ||
> +			  (jit_enabled && !prog_infos[i].jited_prog_len) ||
>   			  !prog_infos[i].xlated_prog_len,
>   			  "get-prog-info(fd)",
> -			  "err %d errno %d i %d type %d(%d) info_len %u(%lu) jited_prog_len %u xlated_prog_len %u\n",
> +			  "err %d errno %d i %d type %d(%d) info_len %u(%lu) jit_enabled %d jited_prog_len %u xlated_prog_len %u\n",
>   			  err, errno, i,
>   			  prog_infos[i].type, BPF_PROG_TYPE_SOCKET_FILTER,
>   			  info_len, sizeof(struct bpf_prog_info),
> +			  jit_enabled,
>   			  prog_infos[i].jited_prog_len,
>   			  prog_infos[i].xlated_prog_len))
>   			goto done;
> 
Acked-by: Yonghong Song <yhs@...com>