lkml.org 
[lkml]   [2014]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.17 107/141] nfs: Fix use of uninitialized variable in nfs_getattr()
    Date
    3.17-stable review patch.  If anyone has any objections, please let me know.

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

    From: Jan Kara <jack@suse.cz>

    commit 16caf5b6101d03335b386e77e9e14136f989be87 upstream.

    Variable 'err' needn't be initialized when nfs_getattr() uses it to
    check whether it should call generic_fillattr() or not. That can result
    in spurious error returns. Initialize 'err' properly.

    Signed-off-by: Jan Kara <jack@suse.cz>
    Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

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

    --- a/fs/nfs/inode.c
    +++ b/fs/nfs/inode.c
    @@ -624,7 +624,7 @@ int nfs_getattr(struct vfsmount *mnt, st
    {
    struct inode *inode = dentry->d_inode;
    int need_atime = NFS_I(inode)->cache_validity & NFS_INO_INVALID_ATIME;
    - int err;
    + int err = 0;

    trace_nfs_getattr_enter(inode);
    /* Flush out writes to the server in order to update c/mtime. */



    \
     
     \ /
      Last update: 2014-11-19 22:41    [W:4.048 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site