diff options
| author | Catalin Marinas <catalin.marinas@arm.com> | 2024-03-12 20:00:38 +0000 |
|---|---|---|
| committer | Catalin Marinas <catalin.marinas@arm.com> | 2024-03-13 10:59:38 +0000 |
| commit | 1ef21fcd6a50f011680dbbd678c1bea8e3f67ab9 (patch) | |
| tree | 489e991dea81ec4f46e3c9404bd08034cc209760 | |
| parent | 69ebc0182406541f0be0f086cdfff13ac56e7385 (diff) | |
Revert "mm: add arch hook to validate mmap() prot flags"
This reverts commit cb1a393c40eee2f1692c995ea0cc6e45bfccde4d.
Since the arm64 WXN patch has been reverted, remove this hook as it
would not have any users.
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Link: https://lore.kernel.org/r/ZfGESD3a91lxH367@arm.com
| -rw-r--r-- | include/linux/mman.h | 15 | ||||
| -rw-r--r-- | mm/mmap.c | 3 |
2 files changed, 0 insertions, 18 deletions
diff --git a/include/linux/mman.h b/include/linux/mman.h index ec5e7f606e43..dc7048824be8 100644 --- a/include/linux/mman.h +++ b/include/linux/mman.h @@ -124,21 +124,6 @@ static inline bool arch_validate_flags(unsigned long flags) #define arch_validate_flags arch_validate_flags #endif -#ifndef arch_validate_mmap_prot -/* - * This is called from mmap(), which ignores unknown prot bits so the default - * is to accept anything. - * - * Returns true if the prot flags are valid - */ -static inline bool arch_validate_mmap_prot(unsigned long prot, - unsigned long addr) -{ - return true; -} -#define arch_validate_mmap_prot arch_validate_mmap_prot -#endif - /* * Optimisation macro. It is equivalent to: * (x & bit1) ? bit2 : 0 diff --git a/mm/mmap.c b/mm/mmap.c index 977a8c3fd9f5..d89770eaab6b 100644 --- a/mm/mmap.c +++ b/mm/mmap.c @@ -1229,9 +1229,6 @@ unsigned long do_mmap(struct file *file, unsigned long addr, if (!(file && path_noexec(&file->f_path))) prot |= PROT_EXEC; - if (!arch_validate_mmap_prot(prot, addr)) - return -EACCES; - /* force arch specific MAP_FIXED handling in get_unmapped_area */ if (flags & MAP_FIXED_NOREPLACE) flags |= MAP_FIXED; |