diff options
| author | Steve French <sfrench@us.ibm.com> | 2007-02-01 04:27:59 +0000 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2007-02-01 04:27:59 +0000 |
| commit | 030e9d8147491a9d2fe1b67882a3720fcf8b95f7 (patch) | |
| tree | 97a68614bd5711e14451964c18660dc4e35109fe | |
| parent | 190ff5b3a168b666925897558998b5d97fec8731 (diff) | |
| download | kernel_samsung_smdk4412-030e9d8147491a9d2fe1b67882a3720fcf8b95f7.zip kernel_samsung_smdk4412-030e9d8147491a9d2fe1b67882a3720fcf8b95f7.tar.gz kernel_samsung_smdk4412-030e9d8147491a9d2fe1b67882a3720fcf8b95f7.tar.bz2 | |
[CIFS] lseek polling returned stale EOF
Fixes Samba bug 4362
Discovered by Jeremy Allison
Clipper database polls on EOF via lseek and can get stale EOF
when file is open on different client
Signed-off-by: Jeremy Allison <jra@samba.org>
Signed-off-by: Steve French <sfrench@us.ibm.com>
| -rw-r--r-- | fs/cifs/cifsfs.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c index 10c9029..93ef099 100644 --- a/fs/cifs/cifsfs.c +++ b/fs/cifs/cifsfs.c @@ -511,7 +511,15 @@ static loff_t cifs_llseek(struct file *file, loff_t offset, int origin) { /* origin == SEEK_END => we must revalidate the cached file length */ if (origin == SEEK_END) { - int retval = cifs_revalidate(file->f_path.dentry); + int retval; + + /* some applications poll for the file length in this strange + way so we must seek to end on non-oplocked files by + setting the revalidate time to zero */ + if(file->f_path.dentry->d_inode) + CIFS_I(file->f_path.dentry->d_inode)->time = 0; + + retval = cifs_revalidate(file->f_path.dentry); if (retval < 0) return (loff_t)retval; } |