bpf: Remove minimum size restrictions on verifier log buffer
authorAndrii Nakryiko <andrii@kernel.org>
Thu, 6 Apr 2023 23:41:48 +0000 (16:41 -0700)
committerDaniel Borkmann <daniel@iogearbox.net>
Tue, 11 Apr 2023 16:05:42 +0000 (18:05 +0200)
It's not clear why we have 128 as minimum size, but it makes testing
harder and seems unnecessary, as we carefully handle truncation
scenarios and use proper snprintf variants. So remove this limitation
and just enforce positive length for log buffer.

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-3-andrii@kernel.org
kernel/bpf/log.c

index 920061e38d2e1b6256d851aef2780f0db381e930..1974891fc324b017bd99b9c817db122ee70249b0 100644 (file)
@@ -11,7 +11,7 @@
 
 bool bpf_verifier_log_attr_valid(const struct bpf_verifier_log *log)
 {
-       return log->len_total >= 128 && log->len_total <= UINT_MAX >> 2 &&
+       return log->len_total > 0 && log->len_total <= UINT_MAX >> 2 &&
               log->level && log->ubuf && !(log->level & ~BPF_LOG_MASK);
 }