lkml.org 
[lkml]   [2012]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[26/75] NFS: Fix a regression in nfs_file_llseek()
    3.1-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Trond Myklebust <Trond.Myklebust@netapp.com>

    commit 6c52961743f38747401b47127b82159ab6d8a7a4 upstream.

    After commit 06222e491e663dac939f04b125c9dc52126a75c4 (fs: handle
    SEEK_HOLE/SEEK_DATA properly in all fs's that define their own llseek)
    the behaviour of llseek() was changed so that it always revalidates
    the file size. The bug appears to be due to a logic error in the
    afore-mentioned commit, which always evaluates to 'true'.

    Reported-by: Roel Kluin <roel.kluin@gmail.com>
    Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    fs/nfs/file.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    --- a/fs/nfs/file.c
    +++ b/fs/nfs/file.c
    @@ -191,7 +191,7 @@ static loff_t nfs_file_llseek(struct fil
    * origin == SEEK_END || SEEK_DATA || SEEK_HOLE => we must revalidate
    * the cached file length
    */
    - if (origin != SEEK_SET || origin != SEEK_CUR) {
    + if (origin != SEEK_SET && origin != SEEK_CUR) {
    struct inode *inode = filp->f_mapping->host;

    int retval = nfs_revalidate_file_size(inode, filp);



    \
     
     \ /
      Last update: 2012-01-04 00:07    [W:4.106 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site