]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/commit
vfs: Don't let __fdget_pos() get FMODE_PATH files
authorEric Biggers <ebiggers3@gmail.com>
Sun, 16 Mar 2014 20:47:48 +0000 (15:47 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 23 Mar 2014 04:03:12 +0000 (00:03 -0400)
commit99aea68134f3c2a27b4d463c91cfa298c3efaccf
tree6b57fe890caaa2ff026a9d54a42f9ac95b96fee9
parentd7a15f8d0777955986a2ab00ab181795cab14b01
vfs: Don't let __fdget_pos() get FMODE_PATH files

Commit bd2a31d522344 ("get rid of fget_light()") introduced the
__fdget_pos() function, which returns the resulting file pointer and
fdput flags combined in an 'unsigned long'.  However, it also changed the
behavior to return files with FMODE_PATH set, which shouldn't happen
because read(), write(), lseek(), etc. aren't allowed on such files.
This commit restores the old behavior.

This regression actually had no effect on read() and write() since
FMODE_READ and FMODE_WRITE are not set on file descriptors opened with
O_PATH, but it did cause lseek() on a file descriptor opened with O_PATH
to fail with ESPIPE rather than EBADF.

Signed-off-by: Eric Biggers <ebiggers3@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/file.c