Linux NFS development
 help / color / mirror / Atom feed
From: Dr Fields James Bruce <bfields@fieldses.org>
To: Trond Myklebust <trond.myklebust@primarydata.com>
Cc: Layton Jeff <jlayton@redhat.com>, NFS <linux-nfs@vger.kernel.org>,
	Adamson William Andros <androsadamson@gmail.com>,
	Lever Charles Edward <chuck.lever@oracle.com>
Subject: Re: v4.0 CB_COMPOUND authentication failures
Date: Tue, 8 Apr 2014 12:40:24 -0400	[thread overview]
Message-ID: <20140408164024.GH3882@fieldses.org> (raw)
In-Reply-To: <ECD8245B-CDEE-442E-A41A-BECE08BEEDA1@primarydata.com>

> 
> > But, I don't know, I'm frankly confused about our security design for
> > the NFSv4 state.
> > 
> > When we insist on krb5 (and checked the server name correctly), and
> > failed without it, then I feel like I understand what we're doing.  Once
> > we start trying it and then falling back (as I understand happens for
> > the krb5 state in the auth_sys case) I get confused.
> 
> Now you have me confused. I’m aware that we call nfs_create_rpc_client() with a krb5i argument and then fall back to auth_sys if the RPC layer says that we don’t have a running gss daemon or that we can’t load the rpcsec_gss_krb5 module. I’m not aware of us falling back if rpc.gssd is running and tells us that security negotiation failed; we should be returning a mount error in that case.

Oh, good, that sounds fine--I'd forgotten it worked that way.

So the problem occurs just because gssd and/or the kerberos libraries
are allowing us to establish state using a different name and then we're
not accepting it on the return.

So:

On Tue, Apr 08, 2014 at 12:22:51PM -0400, Trond Myklebust wrote:
> How is it not better just to rip out that hostname comparison in the
> back channel?

Rip it out entirely?

At that point anyone who can get a credential in the right realm can
send a recall.  RFC made this requirement to prevent that.

But we've already decided we don't care about that in the 4.1 case, so,
hey, maybe.  I guess it wouldn't bother me.

--b.

  reply	other threads:[~2014-04-08 16:40 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-08 12:21 v4.0 CB_COMPOUND authentication failures Jeff Layton
2014-04-08 12:35 ` J. Bruce Fields
2014-04-08 12:42   ` Trond Myklebust
2014-04-08 12:57     ` Dr Fields James Bruce
2014-04-08 13:49   ` Jeff Layton
2014-04-08 14:03     ` J. Bruce Fields
2014-04-08 14:22       ` Jeff Layton
2014-04-08 14:41         ` Jeff Layton
2014-04-08 14:47           ` J. Bruce Fields
2014-04-08 14:23       ` Trond Myklebust
2014-04-08 14:46         ` Dr Fields James Bruce
2014-04-08 15:04           ` Jeff Layton
2014-04-08 15:13             ` Dr Fields James Bruce
2014-04-08 17:25               ` Simo Sorce
2014-04-08 17:28                 ` Jeff Layton
2014-04-08 16:22           ` Trond Myklebust
2014-04-08 16:40             ` Dr Fields James Bruce [this message]
2014-04-08 17:30               ` Trond Myklebust
2014-04-08 17:55                 ` Jeff Layton
2014-04-08 18:03                   ` Trond Myklebust
2014-04-08 18:24                     ` Jeff Layton
2014-04-08 18:45                       ` Trond Myklebust
2014-04-08 18:49                         ` Jeff Layton
2014-04-08 18:03                   ` Dr Fields James Bruce
2014-04-08 16:44             ` Jeff Layton
2014-04-08 17:27               ` Simo Sorce
2014-04-08 17:30                 ` Jeff Layton
2014-04-08 17:39                   ` Frank Filz
2014-04-08 17:59                     ` Jeff Layton
2014-04-08 18:06                     ` Simo Sorce
2014-04-08 22:44                       ` Frank Filz
2014-04-08 22:52                         ` Simo Sorce
2014-04-08 23:31                           ` Frank Filz
2014-04-08 18:01                   ` Simo Sorce
2014-04-08 18:04                     ` Jeff Layton
2014-04-08 18:08                       ` Simo Sorce
2014-04-08 18:11                         ` Dr Fields James Bruce
2014-04-08 18:52                           ` Simo Sorce
2014-04-08 19:01                             ` Trond Myklebust

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140408164024.GH3882@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=androsadamson@gmail.com \
    --cc=chuck.lever@oracle.com \
    --cc=jlayton@redhat.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=trond.myklebust@primarydata.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox