slub: remove impossible condition
authorDavid Rientjes <rientjes@google.com>
Fri, 15 Jan 2010 20:49:56 +0000 (12:49 -0800)
committerPekka Enberg <penberg@cs.helsinki.fi>
Fri, 22 Jan 2010 16:33:36 +0000 (18:33 +0200)
`s' cannot be NULL if kmalloc_caches is not NULL.

This conditional would trigger a NULL pointer on `s', anyway, since it is
immediately derefernced if true.

Acked-by: Christoph Lameter <cl@linux-foundation.org>
Signed-off-by: David Rientjes <rientjes@google.com>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
mm/slub.c

index bddae72f6f49aa3b496bfe3a7efc1a44783e7da1..8fbb2fd70b64b78bb6236d09e8c7128b58a22c22 100644 (file)
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -2641,7 +2641,7 @@ static noinline struct kmem_cache *dma_kmalloc_cache(int index, gfp_t flags)
        if (slab_state >= SYSFS)
                slabflags |= __SYSFS_ADD_DEFERRED;
 
-       if (!s || !text || !kmem_cache_open(s, flags, text,
+       if (!text || !kmem_cache_open(s, flags, text,
                        realsize, ARCH_KMALLOC_MINALIGN, slabflags, NULL)) {
                s->size = 0;
                kfree(text);