mirror of
https://github.com/torvalds/linux.git
synced 2024-12-01 00:21:32 +00:00
mm/usercopy: Drop extra is_vmalloc_or_module() check
Previously virt_addr_valid() was insufficient to validate if virt_to_page() could be called on an address on arm64. This has since been fixed up so there is no need for the extra check. Drop it. Signed-off-by: Laura Abbott <labbott@redhat.com> Acked-by: Mark Rutland <mark.rutland@arm.com> Signed-off-by: Kees Cook <keescook@chromium.org>
This commit is contained in:
parent
96dc4f9fb6
commit
517e1fbeb6
@ -200,17 +200,6 @@ static inline const char *check_heap_object(const void *ptr, unsigned long n,
|
||||
{
|
||||
struct page *page;
|
||||
|
||||
/*
|
||||
* Some architectures (arm64) return true for virt_addr_valid() on
|
||||
* vmalloced addresses. Work around this by checking for vmalloc
|
||||
* first.
|
||||
*
|
||||
* We also need to check for module addresses explicitly since we
|
||||
* may copy static data from modules to userspace
|
||||
*/
|
||||
if (is_vmalloc_or_module_addr(ptr))
|
||||
return NULL;
|
||||
|
||||
if (!virt_addr_valid(ptr))
|
||||
return NULL;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user