All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Christopher J. PeBenito" <cpebenito@tresys.com>
To: Paul Moore <paul@paul-moore.com>
Cc: Stephen Smalley <sds@tycho.nsa.gov>,
	SELinux Mail List <selinux@tycho.nsa.gov>
Subject: Re: RFC: packet checks always on option
Date: Tue, 15 May 2012 11:46:27 -0400	[thread overview]
Message-ID: <4FB27A53.4050903@tresys.com> (raw)
In-Reply-To: <4183776.qZOazSxGrL@sifl>

On 05/15/12 11:04, Paul Moore wrote:
> On Tuesday, May 15, 2012 10:47:25 AM Christopher J. PeBenito wrote:
>> On 05/15/12 10:13, Paul Moore wrote:
>>> On Tuesday, May 15, 2012 09:24:20 AM Christopher J. PeBenito wrote:
>>>> On 05/14/12 17:15, Paul Moore wrote:
>>>>> On Monday, May 14, 2012 01:17:30 PM Stephen Smalley wrote:
>>>>>> I'm not adverse to making it optional/configurable, but I think a
>>>>>> policy
>>>>>> capability is how you should do it.  That is what they are for, and
>>>>>> they
>>>>>> are supposed to provide a more explicit mechanism than either the
>>>>>> handle_unknown logic or the old compat_net logic ...
>>>>>
>>>>> *If* we decide to go this route, I agree, policy capabilities seem to be
>>>>> the best fit.
>>>>>
>>>>> However, as I said earlier in my emails to Chris, I'm still not certain
>>>>> this actually accomplishes anything useful.
>>>>
>>>> I don't understand how you can say this doesn't accomplish anything
>>>> useful.
>>>
>>> See my earlier comments in this thread about being able to verify the
>>> correctness of the secmark labels.  This has always been my core concern
>>> with your argument: you are concerned about the ability for policy to
>>> control network traffic labeled via secmark, but you seem to ignore the
>>> issue that there is no mechanism to verify the correctness of the secmark
>>> labels.  Making strong guarantees about the ability to enforce a given
>>> policy without any assurance that the labels are correct seems a bit
>>> silly to me.
>>
>> Believe me, as a policy person, I'd never ignore labeling correctness.  I
>> don't think SECMARK rule correctness has anything to do with this
>> discussion, as this is about the mechanism/enforcement itself.
> 
> Perhaps I'm reading the two sentences above wrong, perhaps I'm thinking about 
> it wrong, or perhaps you didn't write them as intended; but the two sentences 
> above seem to contradict each other in my mind.  I just don't see how you can 
> have enforcement via labels without correct application of the labels 
> themselves.

Of course for a system to work right you need correct enforcement, correct policy, and correct labeling.  My whole argument is about the enforcement.  If you have correct labeling and correct policy but wrong enforcement, its still incorrect.  I'm only trying to argue on the enforcement; label correctness is important, just not for this discussion.

I can see if you're saying that a system a SECMARK ruleset that fails to load would have incorrect labels for packets.  I agree with that.  But this is a similar situation to /dev labeling being initialized by udev.  The enforcement on files in /dev starts as soon as the filesystem is mounted, rather than when the labeling is initialized by udev.  If udev fails, then the enforcement still goes on with the initial labels, which is why we have restrictive initial labels, rather than disabled.

-- 
Chris PeBenito
Tresys Technology, LLC
www.tresys.com | oss.tresys.com

--
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:[~2012-05-15 15:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-08 16:58 RFC: packet checks always on option Christopher J. PeBenito
2012-05-10 20:02 ` Paul Moore
2012-05-14 12:52   ` Christopher J. PeBenito
2012-05-14 15:35     ` Paul Moore
2012-05-14 16:42       ` Chad Hanson
2012-05-14 20:54         ` Paul Moore
2012-05-14 17:17 ` Stephen Smalley
2012-05-14 17:22   ` Stephen Smalley
2012-05-14 21:15   ` Paul Moore
2012-05-15 13:24     ` Christopher J. PeBenito
2012-05-15 14:13       ` Paul Moore
2012-05-15 14:47         ` Christopher J. PeBenito
2012-05-15 15:04           ` Paul Moore
2012-05-15 15:46             ` Christopher J. PeBenito [this message]
2012-05-15 18:45               ` Paul Moore
2012-05-17 14:06                 ` david caplan
2012-05-17 14:42                   ` Paul Moore
2012-05-17 15:31                     ` david caplan
2012-05-17 16:51                       ` Paul Moore
2012-05-16  2:18           ` Daniel J Walsh

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=4FB27A53.4050903@tresys.com \
    --to=cpebenito@tresys.com \
    --cc=paul@paul-moore.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.