diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2022-06-29 15:06:59 +0200 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2022-07-16 09:19:15 -0400 |
commit | 4e3299eaddffd9d7d5b8bae28ad700bb775f02d0 (patch) | |
tree | e9788bc97a081aa7e6a32cb8a1e525c6667cad1c /fs/overlayfs | |
parent | e7478158e1378325907edfdd960eca98a1be405b (diff) | |
download | lwn-4e3299eaddffd9d7d5b8bae28ad700bb775f02d0.tar.gz lwn-4e3299eaddffd9d7d5b8bae28ad700bb775f02d0.zip |
fs: do not compare against ->llseek
Now vfs_llseek() can simply check for FMODE_LSEEK; if it's set,
we know that ->llseek() won't be NULL and if it's not we should
just fail with -ESPIPE.
A couple of other places where we used to check for special
values of ->llseek() (somewhat inconsistently) switched to
checking FMODE_LSEEK.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/overlayfs')
-rw-r--r-- | fs/overlayfs/copy_up.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/overlayfs/copy_up.c b/fs/overlayfs/copy_up.c index 714ec569d25b..022606c9703b 100644 --- a/fs/overlayfs/copy_up.c +++ b/fs/overlayfs/copy_up.c @@ -226,8 +226,7 @@ static int ovl_copy_up_data(struct ovl_fs *ofs, struct path *old, /* Couldn't clone, so now we try to copy the data */ /* Check if lower fs supports seek operation */ - if (old_file->f_mode & FMODE_LSEEK && - old_file->f_op->llseek) + if (old_file->f_mode & FMODE_LSEEK) skip_hole = true; while (len) { |