diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-07-17 21:52:41 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-07-19 15:09:03 -0400 |
commit | 9936781d0147e13cbc6df02f5265e0b9999ce3cf (patch) | |
tree | a24b01b576cd04f5eb6b9f61da5cfed650d50603 /fs/nfs/nfs4xdr.c | |
parent | 56659e9926ef5996398924c799ecac39ca8b6223 (diff) |
NFSv4: Try to recover from getfh failures in nfs4_xdr_dec_open
Try harder to recover the open state if the server failed to return a
filehandle.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/nfs4xdr.c')
-rw-r--r-- | fs/nfs/nfs4xdr.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/nfs/nfs4xdr.c b/fs/nfs/nfs4xdr.c index 99a123db1083..af959147077b 100644 --- a/fs/nfs/nfs4xdr.c +++ b/fs/nfs/nfs4xdr.c | |||
@@ -3161,11 +3161,12 @@ static int decode_getfh(struct xdr_stream *xdr, struct nfs_fh *fh) | |||
3161 | uint32_t len; | 3161 | uint32_t len; |
3162 | int status; | 3162 | int status; |
3163 | 3163 | ||
3164 | /* Zero handle first to allow comparisons */ | ||
3165 | memset(fh, 0, sizeof(*fh)); | ||
3166 | |||
3164 | status = decode_op_hdr(xdr, OP_GETFH); | 3167 | status = decode_op_hdr(xdr, OP_GETFH); |
3165 | if (status) | 3168 | if (status) |
3166 | return status; | 3169 | return status; |
3167 | /* Zero handle first to allow comparisons */ | ||
3168 | memset(fh, 0, sizeof(*fh)); | ||
3169 | 3170 | ||
3170 | READ_BUF(4); | 3171 | READ_BUF(4); |
3171 | READ32(len); | 3172 | READ32(len); |
@@ -4030,8 +4031,7 @@ static int nfs4_xdr_dec_open(struct rpc_rqst *rqstp, __be32 *p, struct nfs_openr | |||
4030 | status = decode_open(&xdr, res); | 4031 | status = decode_open(&xdr, res); |
4031 | if (status) | 4032 | if (status) |
4032 | goto out; | 4033 | goto out; |
4033 | status = decode_getfh(&xdr, &res->fh); | 4034 | if (decode_getfh(&xdr, &res->fh) != 0) |
4034 | if (status) | ||
4035 | goto out; | 4035 | goto out; |
4036 | if (decode_getfattr(&xdr, res->f_attr, res->server) != 0) | 4036 | if (decode_getfattr(&xdr, res->f_attr, res->server) != 0) |
4037 | goto out; | 4037 | goto out; |