diff options
| author | Chris Mason <chris.mason@oracle.com> | 2009-01-16 16:59:08 (GMT) |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2009-01-16 16:59:08 (GMT) |
| commit | c071fcfdb60e7abbe95e02460005d6bca165bf24 (patch) | |
| tree | 2f7b32f7a554e03e012586ae429ea3bd2794fcb6 /fs/btrfs/super.c | |
| parent | 1d9e2ae949411c2f329f30e01ea0355cd02c4296 (diff) | |
| download | linux-c071fcfdb60e7abbe95e02460005d6bca165bf24.tar.xz | |
Btrfs: fix ioctl arg size (userland incompatible change!)
The structure used to send device in btrfs ioctl calls was not
properly aligned, and so 32 bit ioctls would not work properly on
64 bit kernels.
We could fix this with compat ioctls, but we're just one byte away
and it doesn't make sense at this stage to carry about the compat ioctls
forever at this stage in the project.
This patch brings the ioctl arg up to an evenly aligned 4k.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/super.c')
| -rw-r--r-- | fs/btrfs/super.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index b4c101d9..92c9b54 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -582,7 +582,7 @@ static long btrfs_control_ioctl(struct file *file, unsigned int cmd, { struct btrfs_ioctl_vol_args *vol; struct btrfs_fs_devices *fs_devices; - int ret = 0; + int ret = -ENOTTY; int len; if (!capable(CAP_SYS_ADMIN)) @@ -594,6 +594,7 @@ static long btrfs_control_ioctl(struct file *file, unsigned int cmd, goto out; } len = strnlen(vol->name, BTRFS_PATH_NAME_MAX); + switch (cmd) { case BTRFS_IOC_SCAN_DEV: ret = btrfs_scan_one_device(vol->name, FMODE_READ, |