diff options
| author | Dan Rosenberg <dan.j.rosenberg@gmail.com> | 2010-05-15 11:27:37 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-05-26 14:29:18 -0700 |
| commit | cca8198578b62538e5d30efd94fd7b50a062dfb1 (patch) | |
| tree | c4d730a33fd936d3f94f989f119a469b4ddf699c | |
| parent | f46c7299a930469f51e30b7be281c33630917244 (diff) | |
| download | lwn-cca8198578b62538e5d30efd94fd7b50a062dfb1.tar.gz lwn-cca8198578b62538e5d30efd94fd7b50a062dfb1.zip | |
Btrfs: check for read permission on src file in the clone ioctl
commit 5dc6416414fb3ec6e2825fd4d20c8bf1d7fe0395 upstream.
The existing code would have allowed you to clone a file that was
only open for writing
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | fs/btrfs/ioctl.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index cdbb054102b9..64f6b2f24232 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -959,12 +959,17 @@ static noinline long btrfs_ioctl_clone(struct file *file, unsigned long srcfd, ret = -EBADF; goto out_drop_write; } + src = src_file->f_dentry->d_inode; ret = -EINVAL; if (src == inode) goto out_fput; + /* the src must be open for reading */ + if (!(src_file->f_mode & FMODE_READ)) + goto out_fput; + ret = -EISDIR; if (S_ISDIR(src->i_mode) || S_ISDIR(inode->i_mode)) goto out_fput; |