On Mon, 2010-12-06 at 20:25 -0500, andros@xxxxxxxxxx wrote:
> From: Andy Adamson <andros@xxxxxxxxxx>
>
> Indicate support for referrals. Do not set any PNFS roles. Check the flags
> returned by the server for validity. Do not use exchange flags from an old
> client ID instance when recovering a client ID.
>
> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> ---
> fs/nfs/nfs4proc.c | 27 +++++++++++++++++++++++----
> 1 files changed, 23 insertions(+), 4 deletions(-)
>
> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
> index 6a653ff..f2d6508 100644
> --- a/fs/nfs/nfs4proc.c
> +++ b/fs/nfs/nfs4proc.c
> @@ -4477,6 +4477,27 @@ int nfs4_proc_fs_locations(struct inode *dir, const struct qstr *name,
>
> #ifdef CONFIG_NFS_V4_1
> /*
> + * Check the exchange flags returned by the server for invalid flags, having
> + * both PNFS and NON_PNFS flags set, and not having one of NON_PNFS, PNFS, or
> + * DS flags set.
> + */
> +static int nfs4_check_cl_exchange_flags(u32 flags)
> +{
> + if (flags & ~EXCHGID4_FLAG_MASK_R)
> + goto out_inval;
> + if ((flags & EXCHGID4_FLAG_USE_PNFS_MDS) &&
> + (flags & EXCHGID4_FLAG_USE_NON_PNFS))
> + goto out_inval;
> + if (!(flags & (EXCHGID4_FLAG_USE_PNFS_MDS |
> + EXCHGID4_FLAG_USE_NON_PNFS |
> + EXCHGID4_FLAG_USE_PNFS_DS)))
> + goto out_inval;
> + return NFS_OK;
> +out_inval:
> + return -NFS4ERR_INVAL;
> +}
> +
> +/*
> * nfs4_proc_exchange_id()
> *
> * Since the clientid has expired, all compounds using sessions
> @@ -4489,7 +4510,7 @@ int nfs4_proc_exchange_id(struct nfs_client *clp, struct rpc_cred *cred)
> nfs4_verifier verifier;
> struct nfs41_exchange_id_args args = {
> .client = clp,
> - .flags = clp->cl_exchange_flags,
> + .flags = EXCHGID4_FLAG_SUPP_MOVED_REFER,
> };
> struct nfs41_exchange_id_res res = {
> .client = clp,
> @@ -4506,9 +4527,6 @@ int nfs4_proc_exchange_id(struct nfs_client *clp, struct rpc_cred *cred)
> dprintk("--> %s\n", __func__);
> BUG_ON(clp == NULL);
>
> - /* Remove server-only flags */
> - args.flags &= ~EXCHGID4_FLAG_CONFIRMED_R;
> -
> p = (u32 *)verifier.data;
> *p++ = htonl((u32)clp->cl_boot_time.tv_sec);
> *p = htonl((u32)clp->cl_boot_time.tv_nsec);
> @@ -4534,6 +4552,7 @@ int nfs4_proc_exchange_id(struct nfs_client *clp, struct rpc_cred *cred)
> break;
> }
>
> + status = nfs4_check_cl_exchange_flags(clp->cl_exchange_flags);
> dprintk("<-- %s status= %d\n", __func__, status);
> return status;
> }
Looks good. Should I queue up for 2.6.38?
--
Trond Myklebust
Linux NFS client maintainer
NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
