Re: [PATCH] nfsd4: use auth_unix unconditionally on backchannel

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

 



On Wed, Mar 28, 2012 at 11:16:49PM +0000, Myklebust, Trond wrote:
> On Wed, 2012-03-28 at 19:09 -0400, J. Bruce Fields wrote:
> > This is a bandaid.
> > 
> > I have a series of patches that actually implement the correct behavior,
> > but that may not quite be ready for 3.4.
> > 
> > --b.
> > 
> > commit 2f026867c76171d26f003b211063ff0562097d5e
> > Author: J. Bruce Fields <bfields@xxxxxxxxxx>
> > Date:   Wed Mar 28 14:18:16 2012 -0400
> > 
> >     nfsd4: use auth_unix unconditionally on backchannel
> >     
> >     This isn't actually correct, but it works with the Linux client, and
> >     agrees with the behavior we used to have before commit 80fc015bdfe.
> 
> Question: does the Linux client ever send you anything other than
> AUTH_SYS credentials for the csa_sec_parms argument in CREATE_SESSION?
> Anything other than that would be a bug, since our client doesn't
> actually support RPCSEC_GSS in the callback channel.

Right, I've never seen anything else, so I think the client's behaving
as expected.

But the server needs to be fixed to deal with the range of possible
csa_sec_parms possibilities regardless.

The only thing I find odd about the client behavior is why it even
bothers with auth_sys when auth_null would work just as well and be even
slightly simpler.

--b.

> 
> >     Later patches will implement the spec-mandated behavior (which is to use
> >     the security parameters explicitly given by the client in create_session
> >     or backchannel_ctl).
> >     
> 
> 
> -- 
> 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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux