Commit 0a061743 authored by Arnd Bergmann's avatar Arnd Bergmann

compat_ioctl: fix FIONREAD on devices

My final cleanup patch for sys_compat_ioctl() introduced a regression on
the FIONREAD ioctl command, which is used for both regular and special
files, but only works on regular files after my patch, as I had missed
the warning that Al Viro put into a comment right above it.

Change it back so it can work on any file again by moving the implementation
to do_vfs_ioctl() instead.

Fixes: 77b90401 ("compat_ioctl: simplify the implementation")
Reported-and-tested-by: default avatarChristian Zigotzky <chzigotzky@xenosoft.de>
Reported-and-tested-by: default avataryouling257 <youling257@gmail.com>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
parent 77b90401
...@@ -523,13 +523,9 @@ static int compat_ioctl_preallocate(struct file *file, int mode, ...@@ -523,13 +523,9 @@ static int compat_ioctl_preallocate(struct file *file, int mode,
static int file_ioctl(struct file *filp, unsigned int cmd, int __user *p) static int file_ioctl(struct file *filp, unsigned int cmd, int __user *p)
{ {
struct inode *inode = file_inode(filp);
switch (cmd) { switch (cmd) {
case FIBMAP: case FIBMAP:
return ioctl_fibmap(filp, p); return ioctl_fibmap(filp, p);
case FIONREAD:
return put_user(i_size_read(inode) - filp->f_pos, p);
case FS_IOC_RESVSP: case FS_IOC_RESVSP:
case FS_IOC_RESVSP64: case FS_IOC_RESVSP64:
return ioctl_preallocate(filp, 0, p); return ioctl_preallocate(filp, 0, p);
...@@ -721,6 +717,13 @@ static int do_vfs_ioctl(struct file *filp, unsigned int fd, ...@@ -721,6 +717,13 @@ static int do_vfs_ioctl(struct file *filp, unsigned int fd,
case FIDEDUPERANGE: case FIDEDUPERANGE:
return ioctl_file_dedupe_range(filp, argp); return ioctl_file_dedupe_range(filp, argp);
case FIONREAD:
if (!S_ISREG(inode->i_mode))
return vfs_ioctl(filp, cmd, arg);
return put_user(i_size_read(inode) - filp->f_pos,
(int __user *)argp);
default: default:
if (S_ISREG(inode->i_mode)) if (S_ISREG(inode->i_mode))
return file_ioctl(filp, cmd, argp); return file_ioctl(filp, cmd, argp);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment