Re: [BUGFIX] NULL pointer dereference in __vm_enough_memory()

[WU Fengguang <wfg@mail.ustc.edu.cn>]

On Sun, Aug 12, 2007 at 04:17:44PM +0100, Alan Cox wrote:
> Try this (it compiles but isnt tested). Its a weekend here, the sun is
> shining, the beach is a short walk, and I have more interesting things to
> do right now 8)

Have a nice day~~~ It works!

> --- linux.vanilla-2.6.23rc1-mm1/mm/mmap.c	2007-07-26 15:02:58.000000000 +0100
> +++ linux-2.6.23rc1-mm1/mm/mmap.c	2007-08-12 13:53:22.000000000 +0100
> @@ -2058,7 +2058,7 @@
>  	if (__vma && __vma->vm_start < vma->vm_end)
>  		return -ENOMEM;
>  	if ((vma->vm_flags & VM_ACCOUNT) &&
> -	     security_vm_enough_memory(vma_pages(vma)))
> +	     security_vm_enough_memory_mm(mm, vma_pages(vma)))
>  		return -ENOMEM;
>  	vma_link(mm, vma, prev, rb_link, rb_parent);
>  	return 0;

Sorry, I overlooked this chunk in int insert_vm_struct(mm, vma), hehe.