aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMykyta Yatsenko <yatsenko@meta.com>2025-06-17 13:15:36 +0100
committerAndrii Nakryiko <andrii@kernel.org>2025-06-17 13:24:31 -0700
commit66ab68c9de89672366fdc474f4f185bb58cecf2d (patch)
tree9b89e91fe7556992cc6d5c43ae3fa1668f7d2de1
parentselftests/bpf: More precise cpu_mitigations state detection (diff)
downloadlinux-66ab68c9de89672366fdc474f4f185bb58cecf2d.tar.gz
linux-66ab68c9de89672366fdc474f4f185bb58cecf2d.zip
selftests/bpf: Fix unintentional switch case fall through
Break from switch expression after parsing -n CLI argument in veristat, instead of falling through and enabling comparison mode. Fixes: a5c57f81eb2b ("veristat: add ability to set BPF_F_TEST_SANITY_STRICT flag with -r flag") Signed-off-by: Mykyta Yatsenko <yatsenko@meta.com> Signed-off-by: Andrii Nakryiko <andrii@kernel.org> Acked-by: Yonghong Song <yonghong.song@linux.dev> Link: https://lore.kernel.org/bpf/20250617121536.1320074-1-mykyta.yatsenko5@gmail.com
-rw-r--r--tools/testing/selftests/bpf/veristat.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/testing/selftests/bpf/veristat.c b/tools/testing/selftests/bpf/veristat.c
index 56771650ee8c..4da627ca5749 100644
--- a/tools/testing/selftests/bpf/veristat.c
+++ b/tools/testing/selftests/bpf/veristat.c
@@ -364,6 +364,7 @@ static error_t parse_arg(int key, char *arg, struct argp_state *state)
fprintf(stderr, "invalid top N specifier: %s\n", arg);
argp_usage(state);
}
+ break;
case 'C':
env.comparison_mode = true;
break;