All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joshua Brindle <method@manicmethod.com>
To: Paul Moore <paul.moore@hp.com>
Cc: Eric Paris <eparis@redhat.com>,
	Stephen Smalley <sds@tycho.nsa.gov>,
	selinux@tycho.nsa.gov, jmorris@namei.org,
	Karl MacMillan <kmacmillan@mentalrootkit.com>
Subject: Re: [PATCH] kernel: selinux: policy selectable handling of unknown classes and perms
Date: Thu, 02 Aug 2007 15:44:33 -0400	[thread overview]
Message-ID: <46B23421.1030101@manicmethod.com> (raw)
In-Reply-To: <200708021443.26314.paul.moore@hp.com>

Paul Moore wrote:
> On Thursday, August 2 2007 1:25:12 pm Eric Paris wrote:
>   
>> Maybe instead a versioning scheme?  Every security check comes with a
>> 'version.'  Every time we add/change a security check we add one to the
>> 'version' for that check.  If the policy handed to the kernel has a
>> 'version' less than the value we get from the security hook call coded
>> in kernel we can just allow it?   Maybe that could be a solution to the
>> issue I am addressing now along with things like this netlabel change?
>> It certainly would have to touch a whole lot of stuff and might be a
>> pain to keep up with.  Lets say I move a security hook around in a
>> function, I need to decide if the version need to be bumped on that hook
>> or not.  Not impossible, but not pretty.  Also probably means a policy
>> version update, which in turn breaks the discussion which we have
>> ongoing above....
>>     
>
> While I'm not an expert on the policy aspects of this discussion and how the 
> kernel and userspace objects managers interact I think at some point we are 
> going to need something like what Eric proposes above.  With the requirement 
> that new kernels must not regress when using old policy we are going to run 
> into issues as we move forward and add new permission checks.  While I don't 
> think we've seen it yet, it is reasonable to think that this problem will 
> expand into userspace as we develop more and more userspace object managers 
> such a X, CUPS, Postgres, etc.
>
> I know the changes aren't likely to be pretty, I'm kinda cringing at the 
> thought actually, but I think it's a necessary evil for the long term 
> survivability of SELinux.  At least that's what I took away from Eric and 
> Stephen's comments on this thread and I tend to agree.
>   

I'd have to give a pretty hard NAK (for what it would be worth) to the 
versioning system, it simply doesn't scale; in the past we've used 
policy versions to decide how some permission checks work (see fine 
grained netlink discussions) and it was fairly non-ideal. I have to go 
back to the 'the object manager should decide how to deal with unknown 
permissions' argument. The object manager knows how its objects are 
used, and how the security system affects those objects, it is in the 
best position to make those decisions, the security server can only make 
a rough estimation on what the object manager and user want to do, which 
may or may not be accurate.

Here is an example, granted kernexec uses the reboot capability but 
suppose we wanted to make it a separate permission, I'm fairly certain 
that we'd want to always deny it if the policy didn't know about it, 
whereas SE-X is completely unusable without policy and still has OS 
level policy determining who can talk to the X daemon itself, SE-X would 
have to make a determination of whether to run at all or run with course 
grained permissions, chances are most people would want the latter. This 
means we'd actually have environments where some object managers deny by 
default and others allow (or even short circuit access attempts 
altogether). This is ideal from a scalability and dynamic system 
perspective and seems by far to be the best solution.


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

  reply	other threads:[~2007-08-02 19:44 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-01 15:49 [PATCH] kernel: selinux: policy selectable handling of unknown classes and perms Eric Paris
2007-08-02 14:39 ` Stephen Smalley
2007-08-02 15:18   ` Joshua Brindle
2007-08-02 15:33     ` Stephen Smalley
2007-08-02 17:06       ` Joshua Brindle
2007-08-02 17:25       ` Eric Paris
2007-08-02 18:43         ` Paul Moore
2007-08-02 19:44           ` Joshua Brindle [this message]
2007-08-02 20:06             ` Stephen Smalley
2007-08-02 21:05               ` Joshua Brindle
2007-08-02 20:25             ` Eamon Walsh
2007-08-02 21:53               ` Daniel J Walsh
2007-08-02 19:58         ` Stephen Smalley
2007-08-09 19:19 ` Stephen Smalley
2007-08-21 19:40 ` Stephen Smalley

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=46B23421.1030101@manicmethod.com \
    --to=method@manicmethod.com \
    --cc=eparis@redhat.com \
    --cc=jmorris@namei.org \
    --cc=kmacmillan@mentalrootkit.com \
    --cc=paul.moore@hp.com \
    --cc=sds@tycho.nsa.gov \
    --cc=selinux@tycho.nsa.gov \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.