]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
selftests/bpf: Fix unintentional switch case fall through
authorMykyta Yatsenko <yatsenko@meta.com>
Tue, 17 Jun 2025 12:15:36 +0000 (13:15 +0100)
committerAndrii Nakryiko <andrii@kernel.org>
Tue, 17 Jun 2025 20:24:31 +0000 (13:24 -0700)
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
tools/testing/selftests/bpf/veristat.c

index 56771650ee8cd7aa95e04156aec3aaa9b3477041..4da627ca57496905e63a64db0aea6b3ae1242b13 100644 (file)
@@ -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;