diff options
| author | Trond Myklebust | 2012-06-05 15:16:47 +0200 |
|---|---|---|
| committer | Trond Myklebust | 2012-06-05 16:00:14 +0200 |
| commit | 1549210fcc17e9ae20c09ac8cd4c48a8dfd431bd (patch) | |
| tree | 87db73acf11d00c1c4366e488faf86cd70ed996a /fs/nfs/nfs4_fs.h | |
| parent | NFSv4.1: Fix a request leak on the back channel (diff) | |
| download | kernel-qcow2-linux-1549210fcc17e9ae20c09ac8cd4c48a8dfd431bd.tar.gz kernel-qcow2-linux-1549210fcc17e9ae20c09ac8cd4c48a8dfd431bd.tar.xz kernel-qcow2-linux-1549210fcc17e9ae20c09ac8cd4c48a8dfd431bd.zip | |
NFSv4: Fix an Oops in the open recovery code
The open recovery code does not need to request a new value for the
mdsthreshold, and so does not allocate a struct nfs4_threshold.
The problem is that encode_getfattr_open() will still request an
mdsthreshold, and so we end up Oopsing in decode_attr_mdsthreshold.
This patch fixes encode_getfattr_open so that it doesn't request an
mdsthreshold when the caller isn't asking for one. It also fixes
decode_attr_mdsthreshold so that it errors if the server returns
an mdsthreshold that we didn't ask for (instead of Oopsing).
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Andy Adamson <andros@netapp.com>
Diffstat (limited to 'fs/nfs/nfs4_fs.h')
| -rw-r--r-- | fs/nfs/nfs4_fs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfs/nfs4_fs.h b/fs/nfs/nfs4_fs.h index c6827f93ab57..cc5900ac61b5 100644 --- a/fs/nfs/nfs4_fs.h +++ b/fs/nfs/nfs4_fs.h @@ -295,7 +295,7 @@ is_ds_client(struct nfs_client *clp) extern const struct nfs4_minor_version_ops *nfs_v4_minor_ops[]; -extern const u32 nfs4_fattr_bitmap[2]; +extern const u32 nfs4_fattr_bitmap[3]; extern const u32 nfs4_statfs_bitmap[2]; extern const u32 nfs4_pathconf_bitmap[2]; extern const u32 nfs4_fsinfo_bitmap[3]; |