diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2011-12-07 20:08:40 -0500 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2011-12-08 08:55:46 -0500 |
| commit | a5d16333612718569ffd26064270e535cb9c3928 (patch) | |
| tree | 8c91371ea5f77630ce3683bf4971bd3e6012cd0c | |
| parent | 274bd4fb3ed6b72c1d77ef8850511f09fc6b8e4d (diff) | |
| download | linux-a5d16333612718569ffd26064270e535cb9c3928.tar.gz linux-a5d16333612718569ffd26064270e535cb9c3928.tar.bz2 linux-a5d16333612718569ffd26064270e535cb9c3928.zip | |
Btrfs: check if the to-be-added device is writable
If we call ioctl(BTRFS_IOC_ADD_DEV) directly, we'll succeed in adding
a readonly device to a btrfs filesystem, and btrfs will write to
that device, emitting kernel errors:
[ 3109.833692] lost page write due to I/O error on loop2
[ 3109.833720] lost page write due to I/O error on loop2
...
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
| -rw-r--r-- | fs/btrfs/volumes.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index c37433d3cd82..0a8c8f8304b1 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c @@ -1611,7 +1611,7 @@ int btrfs_init_new_device(struct btrfs_root *root, char *device_path) if ((sb->s_flags & MS_RDONLY) && !root->fs_info->fs_devices->seeding) return -EINVAL; - bdev = blkdev_get_by_path(device_path, FMODE_EXCL, + bdev = blkdev_get_by_path(device_path, FMODE_WRITE | FMODE_EXCL, root->fs_info->bdev_holder); if (IS_ERR(bdev)) return PTR_ERR(bdev); |