A subtest setup can fail in a wide variety of ways, so make sure not to
run it if an issue occurs during its setup. The return value is
already representing whether the setup succeeds or fails, it is just
about wiring it.

Signed-off-by: Alexis LothorĂ© (eBPF Foundation) <[email protected]>
---
 tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c 
b/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
index cf2e088bfe8e..1d8d38e67f8b 100644
--- a/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
+++ b/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
@@ -666,8 +666,8 @@ void test_tc_tunnel(void)
                ret = build_subtest_name(cfg, cfg->name, TEST_NAME_MAX_LEN);
                if (ret < 0 || !test__start_subtest(cfg->name))
                        continue;
-               subtest_setup(skel, cfg);
-               run_test(cfg);
+               if (subtest_setup(skel, cfg) == 0)
+                       run_test(cfg);
                subtest_cleanup(cfg);
        }
        cleanup();

-- 
2.51.1.dirty


Reply via email to