mm: initialize kmem_cache pointer to NULL
Subsystems (2)
PATCH 1/1 mm: initialize kmem_cache pointer to NULL
The assignment to NULL within the error condition was written in a 2014 patch to suppress a compiler warning. However it would be cleaner to just initialize the kmem_cache to NULL and just return it in case of an error condition.
--- mm/slab_common.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
mm/slab_common.c +1 -2
@@ -384,7 +384,7 @@ struct kmem_cache * kmem_cache_create(const char *name, size_t size, size_t align, unsigned long flags, void (*ctor)(void *)) { - struct kmem_cache *s; + struct kmem_cache *s = NULL;const char *cache_name;int err;
@@ -396,7 +396,6 @@ kmem_cache_create(const char *name, size_t size, size_t align, err =kmem_cache_sanity_check (name, size);if (err) { - s =NULL ;/* suppress uninit var warning */ goto out_unlock; }
Signature · 1 line
2.6.1
Quoted 4 lines · Alexandru Moise
The assignment to NULL within the error condition was written in a 2014 patch to suppress a compiler warning. However it would be cleaner to just initialize the
kmem_cacheto NULL and just return it in case of an error condition.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>
Quoted 6 lines · Alexandru Moise
The assignment to NULL within the error condition was written in a 2014 patch to suppress a compiler warning. However it would be cleaner to just initialize the
kmem_cacheto NULL and just return it in case of an error condition.Signed-off-by Alexandru Moise <[email protected]>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>