Commit 5d682681 authored by Balasubramani Vivekanandan's avatar Balasubramani Vivekanandan Committed by Linus Torvalds

mm/slub.c: fix wrong address during slab padding restoration

Start address calculated for slab padding restoration was wrong.  Wrong
address would point to some section before padding and could cause
corruption

Link: http://lkml.kernel.org/r/1516604578-4577-1-git-send-email-balasubramani_vivekanandan@mentor.comSigned-off-by: default avatarBalasubramani Vivekanandan <balasubramani_vivekanandan@mentor.com>
Cc: Christoph Lameter <cl@linux.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 84ebb582
...@@ -838,6 +838,7 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page) ...@@ -838,6 +838,7 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page)
u8 *start; u8 *start;
u8 *fault; u8 *fault;
u8 *end; u8 *end;
u8 *pad;
int length; int length;
int remainder; int remainder;
...@@ -851,8 +852,9 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page) ...@@ -851,8 +852,9 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page)
if (!remainder) if (!remainder)
return 1; return 1;
pad = end - remainder;
metadata_access_enable(); metadata_access_enable();
fault = memchr_inv(end - remainder, POISON_INUSE, remainder); fault = memchr_inv(pad, POISON_INUSE, remainder);
metadata_access_disable(); metadata_access_disable();
if (!fault) if (!fault)
return 1; return 1;
...@@ -860,9 +862,9 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page) ...@@ -860,9 +862,9 @@ static int slab_pad_check(struct kmem_cache *s, struct page *page)
end--; end--;
slab_err(s, page, "Padding overwritten. 0x%p-0x%p", fault, end - 1); slab_err(s, page, "Padding overwritten. 0x%p-0x%p", fault, end - 1);
print_section(KERN_ERR, "Padding ", end - remainder, remainder); print_section(KERN_ERR, "Padding ", pad, remainder);
restore_bytes(s, "slab padding", POISON_INUSE, end - remainder, end); restore_bytes(s, "slab padding", POISON_INUSE, fault, end);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment