This one needs be applied with
"nfsd: New counter for generating client confirm verifier"
that is only update client confirm verifier without fix this problem.
thanks,
Kinglong Mee
On 7/13/2015 17:29, Kinglong Mee wrote:
> Commit 294ac32e99 "nfsd: protect clid and verifier generation with client_lock"
> have moved gen_confirm() to gen_clid().
>
> After it, setclientid will return a bad reply with all zero confirms
> after copy_clid().
>
> Signed-off-by: Kinglong Mee <kinglongmee@xxxxxxxxx>
> ---
> fs/nfsd/nfs4state.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> index e0a4556..b1f84fc 100644
> --- a/fs/nfsd/nfs4state.c
> +++ b/fs/nfsd/nfs4state.c
> @@ -3042,10 +3042,11 @@ nfsd4_setclientid(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
> unconf = find_unconfirmed_client_by_name(&clname, nn);
> if (unconf)
> unhash_client_locked(unconf);
> - if (conf && same_verf(&conf->cl_verifier, &clverifier))
> + if (conf && same_verf(&conf->cl_verifier, &clverifier)) {
> /* case 1: probable callback update */
> copy_clid(new, conf);
> - else /* case 4 (new client) or cases 2, 3 (client reboot): */
> + gen_confirm(new, nn);
> + } else /* case 4 (new client) or cases 2, 3 (client reboot): */
> gen_clid(new, nn);
> new->cl_minorversion = 0;
> gen_callback(new, setclid, rqstp);
>
--
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
