diff options
| author | Sage Weil <sage@inktank.com> | 2012-09-24 21:04:57 -0700 |
|---|---|---|
| committer | Alex Elder <elder@inktank.com> | 2012-10-03 10:51:17 -0500 |
| commit | 457712a0bc5389b75d2c93840a684fd77df2aabb (patch) | |
| tree | 465703170a567b747a1e642a6a80332e0bdbf1f1 | |
| parent | 6cae3717cddaf8e5e96e304733dca66e40d56f89 (diff) | |
| download | linux-457712a0bc5389b75d2c93840a684fd77df2aabb.tar.gz linux-457712a0bc5389b75d2c93840a684fd77df2aabb.tar.bz2 linux-457712a0bc5389b75d2c93840a684fd77df2aabb.zip | |
ceph: return EIO on invalid layout on GET_DATALOC ioctl
If the user calls GET_DATALOC on a file with an invalid (e.g.,
zeroed) layout, return EIO to userland.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Alex Elder <elder@inktank.com>
| -rw-r--r-- | fs/ceph/ioctl.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/fs/ceph/ioctl.c b/fs/ceph/ioctl.c index 1396ceb46797..36549a46e311 100644 --- a/fs/ceph/ioctl.c +++ b/fs/ceph/ioctl.c @@ -187,14 +187,18 @@ static long ceph_ioctl_get_dataloc(struct file *file, void __user *arg) u64 tmp; struct ceph_object_layout ol; struct ceph_pg pgid; + int r; /* copy and validate */ if (copy_from_user(&dl, arg, sizeof(dl))) return -EFAULT; down_read(&osdc->map_sem); - ceph_calc_file_object_mapping(&ci->i_layout, dl.file_offset, &len, - &dl.object_no, &dl.object_offset, &olen); + r = ceph_calc_file_object_mapping(&ci->i_layout, dl.file_offset, &len, + &dl.object_no, &dl.object_offset, + &olen); + if (r < 0) + return -EIO; dl.file_offset -= dl.object_offset; dl.object_size = ceph_file_layout_object_size(ci->i_layout); dl.block_size = ceph_file_layout_su(ci->i_layout); |