diff options
| author | Alex Elder <elder@inktank.com> | 2012-08-10 13:12:07 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2012-09-21 20:48:54 -0700 |
| commit | 340c7a2b2c9a2da640af28a8c196356484ac8b50 (patch) | |
| tree | 4bd52ff7fc913c1b9bcb35e13c4df3edbb8067df | |
| parent | 5698bd757d55b1bb87edd1a9744ab09c142abfc2 (diff) | |
| download | op-kernel-dev-340c7a2b2c9a2da640af28a8c196356484ac8b50.zip op-kernel-dev-340c7a2b2c9a2da640af28a8c196356484ac8b50.tar.gz | |
rbd: drop dev reference on error in rbd_open()
If a read-only rbd device is opened for writing in rbd_open(), it
returns without dropping the just-acquired device reference.
Fix this by moving the read-only check before getting the reference.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
| -rw-r--r-- | drivers/block/rbd.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 9917943..54a55f0 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -246,13 +246,12 @@ static int rbd_open(struct block_device *bdev, fmode_t mode) { struct rbd_device *rbd_dev = bdev->bd_disk->private_data; - rbd_get_dev(rbd_dev); - - set_device_ro(bdev, rbd_dev->read_only); - if ((mode & FMODE_WRITE) && rbd_dev->read_only) return -EROFS; + rbd_get_dev(rbd_dev); + set_device_ro(bdev, rbd_dev->read_only); + return 0; } |
