Skip to content

Commit 4dd8694

Browse files
mykyta5gregkh
authored andcommitted
selftests/bpf: Fix unintentional switch case fall through
[ Upstream commit 66ab68c ] Break from switch expression after parsing -n CLI argument in veristat, instead of falling through and enabling comparison mode. Fixes: a5c57f8 ("veristat: add ability to set BPF_F_TEST_SANITY_STRICT flag with -r flag") Signed-off-by: Mykyta Yatsenko <[email protected]> Signed-off-by: Andrii Nakryiko <[email protected]> Acked-by: Yonghong Song <[email protected]> Link: https://lore.kernel.org/bpf/[email protected] Signed-off-by: Sasha Levin <[email protected]>
1 parent 8e5644c commit 4dd8694

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tools/testing/selftests/bpf/veristat.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -344,6 +344,7 @@ static error_t parse_arg(int key, char *arg, struct argp_state *state)
344344
fprintf(stderr, "invalid top N specifier: %s\n", arg);
345345
argp_usage(state);
346346
}
347+
break;
347348
case 'C':
348349
env.comparison_mode = true;
349350
break;

0 commit comments

Comments
 (0)