diff options
author | NeilBrown <neilb@suse.de> | 2006-01-06 03:20:40 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-06 11:34:07 -0500 |
commit | 1345b1d8adbdeceb1c871d9a4af5e2a700b341c6 (patch) | |
tree | 0da1db7d273752ad5281db415e9245f3f3a63383 /drivers/md/raid6main.c | |
parent | 7dd5d34c6c2da04e8cd0732e08b33cd3359e4bae (diff) |
[PATCH] md: define and use safe_put_page for md
md sometimes call put_page on NULL pointers (treating it like kfree). This is
not safe, so define and use a 'safe_put_page' which checks for NULL.
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/md/raid6main.c')
-rw-r--r-- | drivers/md/raid6main.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/md/raid6main.c b/drivers/md/raid6main.c index 950e5fa6e1f2..06b32bd671a3 100644 --- a/drivers/md/raid6main.c +++ b/drivers/md/raid6main.c | |||
@@ -2072,8 +2072,7 @@ static int run(mddev_t *mddev) | |||
2072 | abort: | 2072 | abort: |
2073 | if (conf) { | 2073 | if (conf) { |
2074 | print_raid6_conf(conf); | 2074 | print_raid6_conf(conf); |
2075 | if (conf->spare_page) | 2075 | safe_put_page(conf->spare_page); |
2076 | put_page(conf->spare_page); | ||
2077 | kfree(conf->stripe_hashtbl); | 2076 | kfree(conf->stripe_hashtbl); |
2078 | kfree(conf); | 2077 | kfree(conf); |
2079 | } | 2078 | } |