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.