diff options
author | Andrii Nakryiko <andrii@kernel.org> | 2024-10-17 20:47:13 +0300 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2024-10-17 22:30:32 +0300 |
commit | 5ac9b4e935dfc6af41eee2ddc21deb5c36507a9f (patch) | |
tree | 0774b4530b66c6edbe75617512fbfda41f1def3d /kernel/trace | |
parent | db123e42304d5bba9a1e04341db0cafaa7a52f35 (diff) | |
download | linux-5ac9b4e935dfc6af41eee2ddc21deb5c36507a9f.tar.xz |
lib/buildid: Handle memfd_secret() files in build_id_parse()
>From memfd_secret(2) manpage:
The memory areas backing the file created with memfd_secret(2) are
visible only to the processes that have access to the file descriptor.
The memory region is removed from the kernel page tables and only the
page tables of the processes holding the file descriptor map the
corresponding physical memory. (Thus, the pages in the region can't be
accessed by the kernel itself, so that, for example, pointers to the
region can't be passed to system calls.)
We need to handle this special case gracefully in build ID fetching
code. Return -EFAULT whenever secretmem file is passed to build_id_parse()
family of APIs. Original report and repro can be found in [0].
[0] https://lore.kernel.org/bpf/ZwyG8Uro%2FSyTXAni@ly-workstation/
Fixes: de3ec364c3c3 ("lib/buildid: add single folio-based file reader abstraction")
Reported-by: Yi Lai <yi1.lai@intel.com>
Suggested-by: Shakeel Butt <shakeel.butt@linux.dev>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Shakeel Butt <shakeel.butt@linux.dev>
Link: https://lore.kernel.org/bpf/20241017175431.6183-A-hca@linux.ibm.com
Link: https://lore.kernel.org/bpf/20241017174713.2157873-1-andrii@kernel.org
Diffstat (limited to 'kernel/trace')
0 files changed, 0 insertions, 0 deletions