diff options
| author | Alexei Starovoitov <ast@kernel.org> | 2023-03-13 16:58:43 -0700 |
|---|---|---|
| committer | Martin KaFai Lau <martin.lau@kernel.org> | 2023-03-13 23:08:21 -0700 |
| commit | c9267aa8b794c2188d49c7d7bd2990e98b2d6b84 (patch) | |
| tree | 7dd80af18ce80a68cd887b86cb2e55ed23870012 | |
| parent | b9fe8e8d03d0df28b2431e3aaf8e115cf7bf2f65 (diff) | |
| download | bpf-c9267aa8b794c2188d49c7d7bd2990e98b2d6b84.tar.gz | |
bpf: Fix bpf_strncmp proto.
bpf_strncmp() doesn't write into its first argument.
Make sure that the verifier knows about it.
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: David Vernet <void@manifault.com>
Link: https://lore.kernel.org/r/20230313235845.61029-2-alexei.starovoitov@gmail.com
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
| -rw-r--r-- | kernel/bpf/helpers.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/bpf/helpers.c b/kernel/bpf/helpers.c index 77d64b6951b9c9..f753676ef65228 100644 --- a/kernel/bpf/helpers.c +++ b/kernel/bpf/helpers.c @@ -571,7 +571,7 @@ static const struct bpf_func_proto bpf_strncmp_proto = { .func = bpf_strncmp, .gpl_only = false, .ret_type = RET_INTEGER, - .arg1_type = ARG_PTR_TO_MEM, + .arg1_type = ARG_PTR_TO_MEM | MEM_RDONLY, .arg2_type = ARG_CONST_SIZE, .arg3_type = ARG_PTR_TO_CONST_STR, }; |