diff options
| author | Jun'ichi Nomura <j-nomura@ce.jp.nec.com> | 2006-10-30 16:23:45 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-30 19:29:41 -0800 |
| commit | bcb55165d3d1ae3ec95807d118fd6d5956cd127b (patch) | |
| tree | 9079f23a4893724a720c3dd0ce3051d476dff43e /fs | |
| parent | a7aacdf9ea45bf6139cfd750e558a3dcbc6f16c3 (diff) | |
| download | linux-bcb55165d3d1ae3ec95807d118fd6d5956cd127b.tar.bz2 | |
[PATCH] fix bd_claim_by_kobject error handling
This fixes bd_claim_by_kobject to release bdev correctly in case that
bd_claim succeeds but following add_bd_holder fails.
Signed-off-by: Jun'ichi Nomura <j-nomura@ce.jp.nec.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/block_dev.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/block_dev.c b/fs/block_dev.c index 702b88cbd91d..b54b0a1b7c68 100644 --- a/fs/block_dev.c +++ b/fs/block_dev.c @@ -751,8 +751,11 @@ static int bd_claim_by_kobject(struct block_device *bdev, void *holder, mutex_lock_nested(&bdev->bd_mutex, BD_MUTEX_PARTITION); res = bd_claim(bdev, holder); - if (res == 0) + if (res == 0) { res = add_bd_holder(bdev, bo); + if (res) + bd_release(bdev); + } if (res) free_bd_holder(bo); mutex_unlock(&bdev->bd_mutex); |