diff options
author | MinChan Kim <minchan.kim@gmail.com> | 2008-05-19 22:12:08 +0900 |
---|---|---|
committer | Pekka Enberg <penberg@cs.helsinki.fi> | 2008-05-19 20:55:25 +0300 |
commit | 239f49c0800778c863585a103805c58afbad6748 (patch) | |
tree | bc49ae2d4405b4431e6e7463b3eac52245793a02 /mm/slob.c | |
parent | f26a3988917913b3d11b2bd741601a2c64ab9204 (diff) | |
download | kernel_samsung_tuna-239f49c0800778c863585a103805c58afbad6748.zip kernel_samsung_tuna-239f49c0800778c863585a103805c58afbad6748.tar.gz kernel_samsung_tuna-239f49c0800778c863585a103805c58afbad6748.tar.bz2 |
slob: Fix to return wrong pointer
Although slob_alloc return NULL, __kmalloc_node returns NULL + align.
Because align always can be changed, it is very hard for debugging
problem of no page if it don't return NULL.
We have to return NULL in case of no page.
[penberg@cs.helsinki.fi: fix formatting as suggested by Matt.]
Acked-by: Matt Mackall <mpm@selenic.com>
Signed-off-by: MinChan Kim <minchan.kim@gmail.com>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Diffstat (limited to 'mm/slob.c')
-rw-r--r-- | mm/slob.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -469,8 +469,9 @@ void *__kmalloc_node(size_t size, gfp_t gfp, int node) return ZERO_SIZE_PTR; m = slob_alloc(size + align, gfp, align, node); - if (m) - *m = size; + if (!m) + return NULL; + *m = size; return (void *)m + align; } else { void *ret; |