]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
bpf: Reject %p% format string in bprintf-like helpers
authorPaul Chaignon <paul.chaignon@gmail.com>
Tue, 1 Jul 2025 19:47:30 +0000 (21:47 +0200)
committerAlexei Starovoitov <ast@kernel.org>
Tue, 1 Jul 2025 22:22:46 +0000 (15:22 -0700)
commitf8242745871f81a3ac37f9f51853d12854fd0b58
treed50b443e6fc4e8351fc05ba09384ac79a43b3955
parent66701750d5565c574af42bef0b789ce0203e3071
bpf: Reject %p% format string in bprintf-like helpers

static const char fmt[] = "%p%";
    bpf_trace_printk(fmt, sizeof(fmt));

The above BPF program isn't rejected and causes a kernel warning at
runtime:

    Please remove unsupported %\x00 in format string
    WARNING: CPU: 1 PID: 7244 at lib/vsprintf.c:2680 format_decode+0x49c/0x5d0

This happens because bpf_bprintf_prepare skips over the second %,
detected as punctuation, while processing %p. This patch fixes it by
not skipping over punctuation. %\x00 is then processed in the next
iteration and rejected.

Reported-by: syzbot+e2c932aec5c8a6e1d31c@syzkaller.appspotmail.com
Fixes: 48cac3f4a96d ("bpf: Implement formatted output helpers with bstr_printf")
Acked-by: Yonghong Song <yonghong.song@linux.dev>
Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
Link: https://lore.kernel.org/r/a0e06cc479faec9e802ae51ba5d66420523251ee.1751395489.git.paul.chaignon@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
kernel/bpf/helpers.c