Re: [PATCH 2/2] sparc: Check VMA range in sparc_validate_prot()

From: Christoph Hellwig
Date: Wed Oct 07 2020 - 08:36:58 EST


> +++ b/arch/sparc/include/asm/mman.h
> @@ -60,31 +60,41 @@ static inline int sparc_validate_prot(unsigned long prot, unsigned long addr,
> if (prot & ~(PROT_READ | PROT_WRITE | PROT_EXEC | PROT_SEM | PROT_ADI))
> return 0;
> if (prot & PROT_ADI) {
> + struct vm_area_struct *vma, *next;
> +

I'd split all the ADI logic into a separate, preferable out of line
helper.

> + /* reached the end of the range without errors? */
> + if (addr+len <= vma->vm_end)

missing whitespaces around the arithmetic operator.