Commit ca21a3e5 authored by Yonghong Song's avatar Yonghong Song Committed by Alexei Starovoitov

selftests/bpf: Fix a few compiler warnings

With clang building selftests/bpf, I hit a few warnings like below:

  .../bpf_iter.c:592:48: warning: variable 'expected_key_c' set but not used [-Wunused-but-set-variable]
  __u32 expected_key_a = 0, expected_key_b = 0, expected_key_c = 0;
                                                ^

  .../bpf_iter.c:688:48: warning: variable 'expected_key_c' set but not used [-Wunused-but-set-variable]
  __u32 expected_key_a = 0, expected_key_b = 0, expected_key_c = 0;
                                                ^

  .../tc_redirect.c:657:6: warning: variable 'target_fd' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
  if (!ASSERT_OK_PTR(nstoken, "setns " NS_FWD))
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  .../tc_redirect.c:743:6: note: uninitialized use occurs here
  if (target_fd >= 0)
      ^~~~~~~~~

Removing unused variables and initializing the previously-uninitialized variable
to ensure these warnings are gone.
Signed-off-by: default avatarYonghong Song <yhs@fb.com>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20210917043343.3711917-1-yhs@fb.com
parent f706f6c6
...@@ -589,7 +589,7 @@ static void test_overflow(bool test_e2big_overflow, bool ret1) ...@@ -589,7 +589,7 @@ static void test_overflow(bool test_e2big_overflow, bool ret1)
static void test_bpf_hash_map(void) static void test_bpf_hash_map(void)
{ {
__u32 expected_key_a = 0, expected_key_b = 0, expected_key_c = 0; __u32 expected_key_a = 0, expected_key_b = 0;
DECLARE_LIBBPF_OPTS(bpf_iter_attach_opts, opts); DECLARE_LIBBPF_OPTS(bpf_iter_attach_opts, opts);
struct bpf_iter_bpf_hash_map *skel; struct bpf_iter_bpf_hash_map *skel;
int err, i, len, map_fd, iter_fd; int err, i, len, map_fd, iter_fd;
...@@ -638,7 +638,6 @@ static void test_bpf_hash_map(void) ...@@ -638,7 +638,6 @@ static void test_bpf_hash_map(void)
val = i + 4; val = i + 4;
expected_key_a += key.a; expected_key_a += key.a;
expected_key_b += key.b; expected_key_b += key.b;
expected_key_c += key.c;
expected_val += val; expected_val += val;
err = bpf_map_update_elem(map_fd, &key, &val, BPF_ANY); err = bpf_map_update_elem(map_fd, &key, &val, BPF_ANY);
...@@ -685,7 +684,7 @@ static void test_bpf_hash_map(void) ...@@ -685,7 +684,7 @@ static void test_bpf_hash_map(void)
static void test_bpf_percpu_hash_map(void) static void test_bpf_percpu_hash_map(void)
{ {
__u32 expected_key_a = 0, expected_key_b = 0, expected_key_c = 0; __u32 expected_key_a = 0, expected_key_b = 0;
DECLARE_LIBBPF_OPTS(bpf_iter_attach_opts, opts); DECLARE_LIBBPF_OPTS(bpf_iter_attach_opts, opts);
struct bpf_iter_bpf_percpu_hash_map *skel; struct bpf_iter_bpf_percpu_hash_map *skel;
int err, i, j, len, map_fd, iter_fd; int err, i, j, len, map_fd, iter_fd;
...@@ -722,7 +721,6 @@ static void test_bpf_percpu_hash_map(void) ...@@ -722,7 +721,6 @@ static void test_bpf_percpu_hash_map(void)
key.c = i + 3; key.c = i + 3;
expected_key_a += key.a; expected_key_a += key.a;
expected_key_b += key.b; expected_key_b += key.b;
expected_key_c += key.c;
for (j = 0; j < bpf_num_possible_cpus(); j++) { for (j = 0; j < bpf_num_possible_cpus(); j++) {
*(__u32 *)(val + j * 8) = i + j; *(__u32 *)(val + j * 8) = i + j;
......
...@@ -633,7 +633,7 @@ static void test_tc_redirect_peer_l3(struct netns_setup_result *setup_result) ...@@ -633,7 +633,7 @@ static void test_tc_redirect_peer_l3(struct netns_setup_result *setup_result)
struct nstoken *nstoken = NULL; struct nstoken *nstoken = NULL;
int err; int err;
int tunnel_pid = -1; int tunnel_pid = -1;
int src_fd, target_fd; int src_fd, target_fd = -1;
int ifindex; int ifindex;
/* Start a L3 TUN/TAP tunnel between the src and dst namespaces. /* Start a L3 TUN/TAP tunnel between the src and dst namespaces.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment