All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Moore <paul.moore@hp.com>
To: "Christopher J. PeBenito" <cpebenito@tresys.com>
Cc: Stephen Smalley <sds@tycho.nsa.gov>,
	Joshua Brindle <method@manicmethod.com>,
	selinux@tycho.nsa.gov
Subject: Re: [PATCH] REFPOL: Add "rogue" Fedora packet class permissions
Date: Fri, 18 Jan 2008 09:11:05 -0500	[thread overview]
Message-ID: <200801180911.05879.paul.moore@hp.com> (raw)
In-Reply-To: <1200663127.4595.23.camel@gorn>

On Friday 18 January 2008 8:32:07 am Christopher J. PeBenito wrote:
> On Thu, 2008-01-17 at 14:33 -0500, Stephen Smalley wrote:
> > The situation is this:
> > - policy shipped in Fedora 6 and later had these flow_in/flow_out
> > permissions defined in their base modules and in their gen_requires.
> > - Paul Moore is adding new permissions to that class in his labeled
> > networking tree that is included in -mm and queued for 2.6.25,
> > - when you try to load those policies into the resulting kernel, the
> > class validation logic rejects the policies,
> > - any policy modules built by any third parties also have these perms
> > defined in their requires and will fail if we remove them from base,
> > - we can't make any changes to the kernel that break existing userspace
> > and policy (which is why handle_unknown is largely useless to us until
> > all legacy distros that predate it are sufficiently dead and gone).
> >
> > This all came up because akpm reported the failure on his FC6 test box
> > with latest -mm.
> >
> > I suggested just using flow_in/flow_out instead of
> > forward_in/forward_out for Paul's new controls so that we don't have any
> > unused permissions, but Paul and Eric want the more precise names.
>
> I strongly agree with Stephen's suggestion.

So, does the "strongly agree" position mean you won't accept the patch adding 
both "flow" and "forward" permissions to the packet class?  I'll reiterate my 
belief that using "flow" instead of "forward" for the new permission checks 
is a mistake which will cause more confusion in the long run than the 
addition of two unused permissions.  However, you hold the key to the policy 
and if changing the permissions to use "flow" is the only way for us to 
enable the new network access controls then I have little choice.

> Do we have a strategy for eventually reclaiming these permissions if we
> don't reuse them right now?

I'm not aware of one, but it is always possible that future work might find a 
use for the packet "flow" permissions.  It's also highly doubtful from where 
I sit now that we'll come even remotely close to hitting the 32 permission 
limit in the packet class.

-- 
paul moore
linux security @ hp

--
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:[~2008-01-18 14:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-17 17:22 [PATCH] REFPOL: Add "rogue" Fedora packet class permissions Paul Moore
2008-01-17 17:30 ` Eric Paris
2008-01-17 19:13 ` Joshua Brindle
2008-01-17 19:23   ` Paul Moore
2008-01-17 19:33   ` Stephen Smalley
2008-01-18 13:32     ` Christopher J. PeBenito
2008-01-18 14:11       ` Paul Moore [this message]
2008-01-18 14:38         ` Christopher J. PeBenito
2008-01-19  3:44           ` Paul Moore
2008-01-18 15:06       ` Eric Paris
2008-01-18 16:52         ` Joshua Brindle

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=200801180911.05879.paul.moore@hp.com \
    --to=paul.moore@hp.com \
    --cc=cpebenito@tresys.com \
    --cc=method@manicmethod.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.