diff options
| author | Jiri Olsa <jolsa@kernel.org> | 2026-06-11 14:42:25 +0300 |
|---|---|---|
| committer | Alexei Starovoitov <ast@kernel.org> | 2026-06-15 03:24:25 +0300 |
| commit | 65d81609e93140d8dd745fd41eb8a195f83ba7cd (patch) | |
| tree | 0bcdf61b42625d2f65ea0f881e5652ee9019cfa7 | |
| parent | 4d87a251d45b4a95eb4c0abcfab809c9f231258a (diff) | |
| download | linux-65d81609e93140d8dd745fd41eb8a195f83ba7cd.tar.xz | |
bpf: Use user_path_at for path resolution in uprobe_multi
Resolve the uprobe_multi user path with user_path_at() instead of copying
the string with strndup_user() and passing it to kern_path(). This removes
the temporary allocation and keeps the lookup logic in one helper.
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Link: https://lore.kernel.org/r/20260611114230.950379-3-jolsa@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
| -rw-r--r-- | kernel/trace/bpf_trace.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/kernel/trace/bpf_trace.c b/kernel/trace/bpf_trace.c index b5a12af2d3f8..f8990bc6b64c 100644 --- a/kernel/trace/bpf_trace.c +++ b/kernel/trace/bpf_trace.c @@ -3227,7 +3227,6 @@ int bpf_uprobe_multi_link_attach(const union bpf_attr *attr, struct bpf_prog *pr unsigned long size; u32 flags, cnt, i; struct path path; - char *name; pid_t pid; int err; @@ -3272,14 +3271,7 @@ int bpf_uprobe_multi_link_attach(const union bpf_attr *attr, struct bpf_prog *pr !access_ok(ucookies, size)) return -EFAULT; - name = strndup_user(upath, PATH_MAX); - if (IS_ERR(name)) { - err = PTR_ERR(name); - return err; - } - - err = kern_path(name, LOOKUP_FOLLOW, &path); - kfree(name); + err = user_path_at(AT_FDCWD, upath, LOOKUP_FOLLOW, &path); if (err) return err; |