bpf: Ignore verifier log reset in BPF_LOG_KERNEL mode

Verifier log position reset is meaningless in BPF_LOG_KERNEL mode, so
just exit early in bpf_vlog_reset() if log->level is BPF_LOG_KERNEL.

This avoid meaningless put_user() into NULL log->ubuf.

Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Lorenz Bauer <lmb@isovalent.com>
Link: https://lore.kernel.org/bpf/20230406234205.323208-8-andrii@kernel.org

diff --git a/kernel/bpf/log.c b/kernel/bpf/log.c
index 92b1c8ad6601e76872142e2174c126a3f715e09b..d99a50f07187d61fe9d6f03bb1a971f06e3878c6 100644 (file)
--- a/kernel/bpf/log.c
+++ b/kernel/bpf/log.c
@@ -106,7 +106,7 @@ void bpf_vlog_reset(struct bpf_verifier_log *log, u64 new_pos)
        if (WARN_ON_ONCE(new_pos > log->end_pos))
                return;
 
-       if (!bpf_verifier_log_needed(log))
+       if (!bpf_verifier_log_needed(log) || log->level == BPF_LOG_KERNEL)
                return;
 
        /* if position to which we reset is beyond current log window,