Commit 39caa091 authored by Joern Engel's avatar Joern Engel Committed by Linus Torvalds

btree: catch NULL value before it does harm

Storing NULL values in the btree is illegal and can lead to memory
corruption and possible other fun as well.  Catch it on insert, instead
of waiting for the inevitable.
Signed-off-by: default avatarJoern Engel <joern@logfs.org>
Signed-off-by: default avatarRoland Dreier <roland@purestorage.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent cbf8ae32
...@@ -509,6 +509,7 @@ static int btree_insert_level(struct btree_head *head, struct btree_geo *geo, ...@@ -509,6 +509,7 @@ static int btree_insert_level(struct btree_head *head, struct btree_geo *geo,
int btree_insert(struct btree_head *head, struct btree_geo *geo, int btree_insert(struct btree_head *head, struct btree_geo *geo,
unsigned long *key, void *val, gfp_t gfp) unsigned long *key, void *val, gfp_t gfp)
{ {
BUG_ON(!val);
return btree_insert_level(head, geo, key, val, 1, gfp); return btree_insert_level(head, geo, key, val, 1, gfp);
} }
EXPORT_SYMBOL_GPL(btree_insert); EXPORT_SYMBOL_GPL(btree_insert);
......
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