]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
selftests/bpf: Fix two minor compilation warnings reported by GCC 4.9
authorAndrii Nakryiko <andriin@fb.com>
Thu, 20 Aug 2020 06:14:10 +0000 (23:14 -0700)
committerDaniel Borkmann <daniel@iogearbox.net>
Thu, 20 Aug 2020 14:45:09 +0000 (16:45 +0200)
GCC 4.9 seems to be more strict in some regards. Fix two minor issue it
reported.

Fixes: 1c1052e0140a ("tools/testing/selftests/bpf: Add self-tests for new helper bpf_get_ns_current_pid_tgid.")
Fixes: 2d7824ffd25c ("selftests: bpf: Add test for sk_assign")
Signed-off-by: Andrii Nakryiko <andriin@fb.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Yonghong Song <yhs@fb.com>
Link: https://lore.kernel.org/bpf/20200820061411.1755905-3-andriin@fb.com
tools/testing/selftests/bpf/prog_tests/sk_assign.c
tools/testing/selftests/bpf/test_current_pid_tgid_new_ns.c

index 47fa04adc1471f7218265a877fd67d74429fdca4..d43038d2b9e15e1847fafb0714504cfb46cdba56 100644 (file)
@@ -268,6 +268,7 @@ void test_sk_assign(void)
        int server = -1;
        int server_map;
        int self_net;
+       int i;
 
        self_net = open(NS_SELF, O_RDONLY);
        if (CHECK_FAIL(self_net < 0)) {
@@ -286,7 +287,7 @@ void test_sk_assign(void)
                goto cleanup;
        }
 
-       for (int i = 0; i < ARRAY_SIZE(tests) && !READ_ONCE(stop); i++) {
+       for (i = 0; i < ARRAY_SIZE(tests) && !READ_ONCE(stop); i++) {
                struct test_sk_cfg *test = &tests[i];
                const struct sockaddr *addr;
                const int zero = 0;
index ed253f252cd02ec73053aa12bb0897df25cbe4ad..ec53b1ef90d2253667f780337a72bb4f91fbeff1 100644 (file)
@@ -156,4 +156,5 @@ cleanup:
                        bpf_object__close(obj);
                }
        }
+       return 0;
 }