Skip to content

Commit

Permalink
mm: SLAB hardened usercopy support
Browse files Browse the repository at this point in the history
Under CONFIG_HARDENED_USERCOPY, this adds object size checking to the
SLAB allocator to catch any copies that may span objects.

Based on code from PaX and grsecurity.

Signed-off-by: Kees Cook <[email protected]>
Tested-by: Valdis Kletnieks <[email protected]>
  • Loading branch information
kees committed Jul 26, 2016
1 parent 97433ea commit 04385fc
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 0 deletions.
1 change: 1 addition & 0 deletions init/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -1758,6 +1758,7 @@ choice

config SLAB
bool "SLAB"
select HAVE_HARDENED_USERCOPY_ALLOCATOR
help
The regular slab allocator that is established and known to work
well in all environments. It organizes cache hot objects in
Expand Down
30 changes: 30 additions & 0 deletions mm/slab.c
Original file line number Diff line number Diff line change
Expand Up @@ -4477,6 +4477,36 @@ static int __init slab_proc_init(void)
module_init(slab_proc_init);
#endif

#ifdef CONFIG_HARDENED_USERCOPY
/*
* Rejects objects that are incorrectly sized.
*
* Returns NULL if check passes, otherwise const char * to name of cache
* to indicate an error.
*/
const char *__check_heap_object(const void *ptr, unsigned long n,
struct page *page)
{
struct kmem_cache *cachep;
unsigned int objnr;
unsigned long offset;

/* Find and validate object. */
cachep = page->slab_cache;
objnr = obj_to_index(cachep, page, (void *)ptr);
BUG_ON(objnr >= cachep->num);

/* Find offset within object. */
offset = ptr - index_to_obj(cachep, page, objnr) - obj_offset(cachep);

/* Allow address range falling entirely within object size. */
if (offset <= cachep->object_size && n <= cachep->object_size - offset)
return NULL;

return cachep->name;
}
#endif /* CONFIG_HARDENED_USERCOPY */

/**
* ksize - get the actual amount of memory allocated for a given object
* @objp: Pointer to the object
Expand Down

0 comments on commit 04385fc

Please sign in to comment.