Re: [PATCH 1/6] nfsd: fix the check for confirmed openowner in nfs4_preprocess_stateid_op

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 28, 2015 at 03:41:15PM +0200, Christoph Hellwig wrote:
> If we find a non-confirmed openowner we jump to exit the function, but do
> not set an error value.  Fix this by factoring out a helper to do the
> check and properly set the error from nfsd4_validate_stateid.

Thanks.  I can't tell if the bug has any practical consequences--a
correct client should never hit this case as far as I can tell.

--b.

> 
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> ---
>  fs/nfsd/nfs4state.c | 21 +++++++++++----------
>  1 file changed, 11 insertions(+), 10 deletions(-)
> 
> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> index 9072964..66067a2 100644
> --- a/fs/nfsd/nfs4state.c
> +++ b/fs/nfsd/nfs4state.c
> @@ -4498,10 +4498,17 @@ static __be32 check_stateid_generation(stateid_t *in, stateid_t *ref, bool has_s
>  	return nfserr_old_stateid;
>  }
>  
> +static __be32 nfsd4_check_openowner_confirmed(struct nfs4_ol_stateid *ols)
> +{
> +	if (ols->st_stateowner->so_is_open_owner &&
> +	    !(openowner(ols->st_stateowner)->oo_flags & NFS4_OO_CONFIRMED))
> +		return nfserr_bad_stateid;
> +	return nfs_ok;
> +}
> +
>  static __be32 nfsd4_validate_stateid(struct nfs4_client *cl, stateid_t *stateid)
>  {
>  	struct nfs4_stid *s;
> -	struct nfs4_ol_stateid *ols;
>  	__be32 status = nfserr_bad_stateid;
>  
>  	if (ZERO_STATEID(stateid) || ONE_STATEID(stateid))
> @@ -4531,13 +4538,7 @@ static __be32 nfsd4_validate_stateid(struct nfs4_client *cl, stateid_t *stateid)
>  		break;
>  	case NFS4_OPEN_STID:
>  	case NFS4_LOCK_STID:
> -		ols = openlockstateid(s);
> -		if (ols->st_stateowner->so_is_open_owner
> -	    			&& !(openowner(ols->st_stateowner)->oo_flags
> -						& NFS4_OO_CONFIRMED))
> -			status = nfserr_bad_stateid;
> -		else
> -			status = nfs_ok;
> +		status = nfsd4_check_openowner_confirmed(openlockstateid(s));
>  		break;
>  	default:
>  		printk("unknown stateid type %x\n", s->sc_type);
> @@ -4629,8 +4630,8 @@ nfs4_preprocess_stateid_op(struct net *net, struct nfsd4_compound_state *cstate,
>  		status = nfs4_check_fh(current_fh, stp);
>  		if (status)
>  			goto out;
> -		if (stp->st_stateowner->so_is_open_owner
> -		    && !(openowner(stp->st_stateowner)->oo_flags & NFS4_OO_CONFIRMED))
> +		status = nfsd4_check_openowner_confirmed(stp);
> +		if (status)
>  			goto out;
>  		status = nfs4_check_openmode(stp, flags);
>  		if (status)
> -- 
> 1.9.1
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]