Commit f07ea10d authored by Chuck Lever's avatar Chuck Lever Committed by J. Bruce Fields
Browse files

NFSD: SETCLIENTID_CONFIRM returns NFS4ERR_CLID_INUSE too often 



According to RFC 3530bis, the only items SETCLIENTID_CONFIRM processing
should be concerned with is the clientid, clientid verifier, and
principal.  The client's IP address is not supposed to be interesting.

And, NFS4ERR_CLID_INUSE is meant only for principal mismatches.

I triggered this logic with a prototype UCS client -- one that
uses the same nfs_client_id4 string for all servers.  The client
mounted our server via its IPv4, then via its IPv6 address.

Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 8dbf28e4

fs/nfsd/nfs4state.c

+2 −12
Original line number Diff line number Diff line
@@ -2214,7 +2214,6 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, 
			 struct nfsd4_compound_state *cstate,

			 struct nfsd4_setclientid_confirm *setclientid_confirm)

{

	struct sockaddr *sa = svc_addr(rqstp);

	struct nfs4_client *conf, *unconf;

	nfs4_verifier confirm = setclientid_confirm->sc_confirm; 

	clientid_t * clid = &setclientid_confirm->sc_clientid;
@@ -2232,17 +2231,12 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, 
	conf = find_confirmed_client(clid);

	unconf = find_unconfirmed_client(clid);



	status = nfserr_clid_inuse;

	if (conf && !rpc_cmp_addr((struct sockaddr *) &conf->cl_addr, sa))

		goto out;

	if (unconf && !rpc_cmp_addr((struct sockaddr *) &unconf->cl_addr, sa))

		goto out;



	/*

	 * section 14.2.34 of RFC 3530 has a description of

	 * SETCLIENTID_CONFIRM request processing consisting

	 * of 4 bullet points, labeled as CASE1 - CASE4 below.

	 */

	status = nfserr_clid_inuse;

	if (conf && unconf && same_verf(&confirm, &unconf->cl_confirm)) {

		/*

		 * RFC 3530 14.2.34 CASE 1:
@@ -2255,7 +2249,6 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, 
			nfsd4_probe_callback(conf);

			expire_client(unconf);

			status = nfs_ok;



		}

	} else if (conf && !unconf) {

		/*
@@ -2297,11 +2290,8 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, 
		 * Client probably hasn't noticed that we rebooted yet.

		 */

		status = nfserr_stale_clientid;

	} else {

		/* check that we have hit one of the cases...*/

		status = nfserr_clid_inuse;

	}

out:



	nfs4_unlock_state();

	return status;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE