On 6/2/26 7:47 AM, Leo Yan wrote:
> ASAN reports stack-buffer-overflow due to the uninitialized op_name.
> 
> Initialize it to fix the issue.
> 
> Fixes: 054b6c7866c7 ("selftests/bpf: Add verifier log tests for BPF_BTF_LOAD 
> command")
> Signed-off-by: Leo Yan <[email protected]>
> ---
>  tools/testing/selftests/bpf/prog_tests/verifier_log.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tools/testing/selftests/bpf/prog_tests/verifier_log.c 
> b/tools/testing/selftests/bpf/prog_tests/verifier_log.c
> index 
> c01c0114af1b7476d27885ce380d90400a4bdc89..4542bb586d723b8fda2faf8261066f16102c60c0
>  100644
> --- a/tools/testing/selftests/bpf/prog_tests/verifier_log.c
> +++ b/tools/testing/selftests/bpf/prog_tests/verifier_log.c
> @@ -317,6 +317,7 @@ static void verif_btf_log_subtest(bool bad_btf)
>       res = load_btf(&opts, true);
>       ASSERT_EQ(res, -ENOSPC, "half_log_fd");
>       ASSERT_EQ(strlen(logs.buf), 24, "log_fixed_25");
> +     strscpy(op_name, "log_fixed", sizeof(op_name));

Acked-by: Ihor Solodrai <[email protected]>

>       ASSERT_STRNEQ(logs.buf, logs.reference, 24, op_name);
>  
>       /* validate rolling verifier log logic: try all variations of log buf
> 


Reply via email to