summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorLi Zefan <lizf@cn.fujitsu.com>2009-03-09 10:42:45 +0100
committerJens Axboe <jens.axboe@oracle.com>2009-03-14 21:06:52 +0100
commit059ea3318c8ede71851a52b4359fbf1ab0cec301 (patch)
tree29a70645e27947d820884b1249d0d7f26b8ea0d8 /fs
parentblock: Add gfp_mask parameter to bio_integrity_clone() (diff)
downloadlinux-059ea3318c8ede71851a52b4359fbf1ab0cec301.tar.xz
linux-059ea3318c8ede71851a52b4359fbf1ab0cec301.zip
block: fix memory leak in bio_clone()
If bio_integrity_clone() fails, bio_clone() returns NULL without freeing the newly allocated bio. Signed-off-by: Li Zefan <lizf@cn.fujitsu.com> Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/bio.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/bio.c b/fs/bio.c
index cf747378b977..d4f06327c810 100644
--- a/fs/bio.c
+++ b/fs/bio.c
@@ -465,8 +465,10 @@ struct bio *bio_clone(struct bio *bio, gfp_t gfp_mask)
ret = bio_integrity_clone(b, bio, gfp_mask, fs_bio_set);
- if (ret < 0)
+ if (ret < 0) {
+ bio_put(b);
return NULL;
+ }
}
return b;