summaryrefslogtreecommitdiffstats
path: root/drivers/block/rbd.c
diff options
context:
space:
mode:
authorIlya Dryomov <ilya.dryomov@inktank.com>2014-09-11 18:49:18 +0400
committerIlya Dryomov <idryomov@redhat.com>2014-10-14 21:03:26 +0400
commit7dd440c9e0711d828442c3e129ab8bcb9aeeac23 (patch)
tree20185208bba6b36327a9bb58b1e9e4cc5cffc5b1 /drivers/block/rbd.c
parent91883cd27c4e2523dd53e2bd6f71a1c5e0be5a38 (diff)
downloadop-kernel-dev-7dd440c9e0711d828442c3e129ab8bcb9aeeac23.zip
op-kernel-dev-7dd440c9e0711d828442c3e129ab8bcb9aeeac23.tar.gz
rbd: do not return -ERANGE on auth failures
Trying to map an image out of a pool for which we don't have an 'x' permission bit fails with -ERANGE from ceph_extract_encoded_string() due to an unsigned vs signed bug. Fix it and get rid of the -EINVAL sink, thus propagating rbd::get_id cls method errors. (I've seen a bunch of unexplained -ERANGE reports, I bet this is it). Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com> Reviewed-by: Alex Elder <elder@linaro.org>
Diffstat (limited to 'drivers/block/rbd.c')
-rw-r--r--drivers/block/rbd.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 4b97baf..ce457db 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -4924,7 +4924,7 @@ static int rbd_dev_image_id(struct rbd_device *rbd_dev)
ret = image_id ? 0 : -ENOMEM;
if (!ret)
rbd_dev->image_format = 1;
- } else if (ret > sizeof (__le32)) {
+ } else if (ret >= 0) {
void *p = response;
image_id = ceph_extract_encoded_string(&p, p + ret,
@@ -4932,8 +4932,6 @@ static int rbd_dev_image_id(struct rbd_device *rbd_dev)
ret = PTR_ERR_OR_ZERO(image_id);
if (!ret)
rbd_dev->image_format = 2;
- } else {
- ret = -EINVAL;
}
if (!ret) {
OpenPOWER on IntegriCloud