On Tue, May 08, 2012 at 06:40:19PM +0400, Vladimir Murzin wrote: > Move check for TASK_SIZE before MAP_FIXED in sparc32's arch_get_unmapped_area(). I would be good to copy relevant parts of your cover letter into each patch. when the patches are applied we looses the cover letter... > > Signed-off-by: Vladimir Murzin <murzin.v@xxxxxxxxx> > --- > arch/sparc/kernel/sys_sparc_32.c | 9 +++++---- > 1 files changed, 5 insertions(+), 4 deletions(-) > > diff --git a/arch/sparc/kernel/sys_sparc_32.c b/arch/sparc/kernel/sys_sparc_32.c > index 42b282f..7bf30de 100644 > --- a/arch/sparc/kernel/sys_sparc_32.c > +++ b/arch/sparc/kernel/sys_sparc_32.c > @@ -40,6 +40,11 @@ unsigned long arch_get_unmapped_area(struct file *filp, unsigned long addr, unsi > { > struct vm_area_struct * vmm; > > + if (len > TASK_SIZE - PAGE_SIZE) > + return -ENOMEM; This is not the same as the check it replaces - what is going on here? Sam -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html