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

From: WU Fengguang
Date: Mon Aug 13 2007 - 08:51:10 EST


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.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/