mm: slab: Verify the nodeid passed to ____cache_alloc_node
authorAaron Tomlin <atomlin@redhat.com>
Fri, 26 Apr 2013 15:15:34 +0000 (16:15 +0100)
committerPekka Enberg <penberg@kernel.org>
Wed, 1 May 2013 07:57:43 +0000 (10:57 +0300)
If the nodeid is > num_online_nodes() this can cause an Oops and a
panic(). The purpose of this patch is to assert if this condition is
true to aid debugging efforts rather than some random NULL pointer
dereference or page fault.

This patch is in response to BZ#42967 [1].  Using VM_BUG_ON so it's used
only when CONFIG_DEBUG_VM is set, given that ____cache_alloc_node() is a
hot code path.

[1]: https://bugzilla.kernel.org/show_bug.cgi?id=42967

Signed-off-by: Aaron Tomlin <atomlin@redhat.com>
Reviewed-by: Rik van Riel <riel@redhat.com>
Acked-by: Christoph Lameter <cl@linux.com>
Acked-by: Rafael Aquini <aquini@redhat.com>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
mm/slab.c

index 910df63268314cc25b6a2db2fd0e6e4160488572..a98f8db93670379bae9bcbcc68a38427dd174fd4 100644 (file)
--- a/mm/slab.c
+++ b/mm/slab.c
@@ -3289,6 +3289,7 @@ static void *____cache_alloc_node(struct kmem_cache *cachep, gfp_t flags,
        void *obj;
        int x;
 
+       VM_BUG_ON(nodeid > num_online_nodes());
        n = cachep->node[nodeid];
        BUG_ON(!n);