diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-07-25 15:37:29 -0400 |
|---|---|---|
| committer | Jonas ABERG <jonas.aberg@stericsson.com> | 2011-10-28 11:14:26 +0200 |
| commit | 9f0e0f4ec5ac79f60d0f75c3f9f9e06f9803aa27 (patch) | |
| tree | 4394e7887960cb4a8ad8253385de18ced4c500e3 /crypto | |
| parent | 06a3e138665b3920fc67b872cdd66cdea6ecd224 (diff) | |
NFSv4: Don't use the delegation->inode in nfs_mark_return_delegation()
commit ed1e6211a0a134ff23592c6f057af982ad5dab52 upstream.
nfs_mark_return_delegation() is usually called without any locking, and
so it is not safe to dereference delegation->inode. Since the inode is
only used to discover the nfs_client anyway, it makes more sense to
have the callers pass a valid pointer to the nfs_server as a parameter.
Reported-by: Ian Kent <raven@themaw.net>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Change-Id: Ia0a265a9fb22ea800d4394ad8308345734f88a6f
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/35657
Tested-by: Per VAHLNE <per.xx.vahlne@stericsson.com>
Reviewed-by: Jonas ABERG <jonas.aberg@stericsson.com>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions