diff options
| author | Joey Gouly <[email protected]> | 2023-03-08 19:04:20 +0000 |
|---|---|---|
| committer | Andrew Morton <[email protected]> | 2023-03-23 17:18:32 -0700 |
| commit | 6bbf1090672673183a98cd6e19de91fa5a319df0 (patch) | |
| tree | 98f9da68ef274991df60a2af2a66e67bb21b29c7 | |
| parent | 12871a154690c52e2ded718b392a3977c114f6c1 (diff) | |
mm: deduplicate error handling for map_deny_write_exec
Patch series "Fixes for MDWE prctl"
These are four small fixes for the recent memory-write-deny-execute prctl
patches [1]. Two reported by Alexey about error handling and two tooling
fixes by Peter.
This patch (of 4):
Commit cc8d1b097de7 ("mmap: clean up mmap_region() unrolling")
deduplicated the error handling, do the same for the return value of
`map_deny_write_exec`.
Link: https://lkml.kernel.org/r/[email protected]
Link: https://lkml.kernel.org/r/[email protected]
Link: https://lore.kernel.org/linux-arm-kernel/[email protected]/ [1]
Fixes: b507808ebce2 ("mm: implement memory-deny-write-execute as a prctl")
Signed-off-by: Joey Gouly <[email protected]>
Reported-by: Alexey Izbyshev <[email protected]>
Link: https://lore.kernel.org/linux-arm-kernel/[email protected]/
Reviewed-by: Catalin Marinas <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Kees Cook <[email protected]>
Cc: nd <[email protected]>
Cc: Peter Xu <[email protected]>
Cc: Shuah Khan <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
| -rw-r--r-- | mm/mmap.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/mm/mmap.c b/mm/mmap.c index 740b54be3ed4..ad499f7b767f 100644 --- a/mm/mmap.c +++ b/mm/mmap.c @@ -2621,12 +2621,7 @@ cannot_expand: if (map_deny_write_exec(vma, vma->vm_flags)) { error = -EACCES; - if (file) - goto close_and_free_vma; - else if (vma->vm_file) - goto unmap_and_free_vma; - else - goto free_vma; + goto close_and_free_vma; } /* Allow architectures to sanity-check the vm_flags */ |