diff options
| author | 2025-10-14 13:20:37 -0700 | |
|---|---|---|
| committer | 2025-10-23 16:24:36 +0200 | |
| commit | dcdd8c061a48de9a041a745fb5330a9aed36a50c (patch) | |
| tree | e176fb8cf32ac9c79f3b67486d1ef3bad45bc14d /tools/perf/scripts/python/bin/ssh:/git@git.zx2c4.com/git: | |
| parent | cxl/trace: Subtract to find an hpa_alias0 in cxl_poison events (diff) | |
selftests/bpf: make arg_parsing.c more robust to crashes
[ Upstream commit e603a342cf7ecd64ef8f36207dfe1caacb9e2583 ]
We started getting a crash in BPF CI, which seems to originate from
test_parse_test_list_file() test and is happening at this line:
ASSERT_OK(strcmp("test_with_spaces", set.tests[0].name), "test 0 name");
One way we can crash there is if set.cnt zero, which is checked for with
ASSERT_EQ() above, but we proceed after this regardless of the outcome.
Instead of crashing, we should bail out with test failure early.
Similarly, if parse_test_list_file() fails, we shouldn't be even looking
at set, so bail even earlier if ASSERT_OK() fails.
Fixes: 64276f01dce8 ("selftests/bpf: Test_progs can read test lists from file")
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Tested-by: Ihor Solodrai <ihor.solodrai@linux.dev>
Link: https://lore.kernel.org/r/20251014202037.72922-1-andrii@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/bin/ssh:/git@git.zx2c4.com/git:')
0 files changed, 0 insertions, 0 deletions
