Re: [PATCH 06/13] selftests: bpf: Add arena ASAN runtime to libarena

["Emil Tsalapatis" <emil@etsalapatis.com>]

On Thu Jan 22, 2026 at 11:58 AM EST, bot+bpf-ci wrote:
>> diff --git a/tools/testing/selftests/bpf/libarena/src/asan.bpf.c b/tools/testing/selftests/bpf/libarena/src/asan.bpf.c
>> new file mode 100644
>> index 000000000000..7d39ee7b68bd
>> --- /dev/null
>> +++ b/tools/testing/selftests/bpf/libarena/src/asan.bpf.c
>
> [ ... ]
>
>> +static __always_inline int asan_report(s8a __arg_arena *addr, size_t sz,
>> +				       bool write)
>> +{
>> +	/* Only report the first ASAN violation. */
>> +	if (asan_reported && asan_report_once)
>> +		return 0;
>> +
>> +	asan_reported = true;
>> +
>> +	asan_violated = (u64)addr;
>> +
>> +	if (asan_emit_stack) {
>> +		arena_stderr("Memory violation for address %p (0x%lx) for %s of size %ld",
>> +				addr, write ? "write" : "read", sz);
>
> The format string has four format specifiers (%p, %lx, %s, %ld) but only
> three arguments are provided (addr, write?"write":"read", sz). Is there
> a missing argument for the (0x%lx) specifier, or should that part of the
> format string be removed?
>

Ack, there is a missing argument - the scalar representation of the
arena address. Will add.

>> +		bpf_stream_print_stack();
>> +	}
>> +
>> +	return 0;
>> +}
>
>
> ---
> AI reviewed your patch. Please fix the bug or email reply why it's not a bug.
> See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md
>
> CI run summary: https://github.com/kernel-patches/bpf/actions/runs/21256973563