From: "Christopher J. PeBenito" <cpebenito@tresys.com>
To: Paul Moore <pmoore@redhat.com>
Cc: SELinux Mail List <selinux@tycho.nsa.gov>
Subject: Re: Clarification of labeled IPsec checks
Date: Mon, 6 May 2013 09:50:04 -0400 [thread overview]
Message-ID: <5187B50C.4020106@tresys.com> (raw)
In-Reply-To: <39078390.44gaRjNkjA@sifl>
On 05/03/13 15:20, Paul Moore wrote:
> On Friday, May 03, 2013 03:11:48 PM Christopher J. PeBenito wrote:
>> I'm doing some spring cleaning on refpolicy, cleaning out some old
>> unused/unnecessary networking permissions. I'm trying to make sure I have
>> the permissions checks straight, since labeled networking isn't common use.
>> For labeled IPsec, we have the following permissions (assuming all policy
>> capabilities are on--assume maximum checks):
>>
>> netif: ingress/egress
>> node: sendto/recvfrom
>> peer: recv
>> association: sendto/recvfrom
>>
>> I'm told that association perms are checked in the following cases:
>>
>> sendto: when a packet leaves the box (legacy only) and when a SA/flow is
>> checked recvfrom: when an incoming packet is queued on a socket (legacy
>> only)
>>
>> Does "legacy only" mean the checks will eventually go away, or is it for a
>> legacy IPsec configuration?
>
> In this case "legacy" refers to a SELinux policy that doesn't have the netpeer
> policy capability enabled.
So with the capability on, you can only get a sendto check for association, never recvfrom? Wouldn't it make more sense to have a peer send permission instead, going forward? That way we can have the something more expected (send/recv for the same class), rather than peer:recv and association:sendto which seems confusing.
> It has been a while since we introduced the netpeer policy capability so I
> imagine we could start a process of deprecating policies that don't enable it.
> We could dump a warning message if someone loads a policy with the netpeer
> capability disabled and then after a few releases (how many?) we could remove
> the legacy bits from the kernel and reject policies which don't have the
> netpeer capability set.
The common case obviously is no labeling, and in that case, there wouldn't be checks. So I suspect this wouldn't have any real problems. My guess is systems that use this wouldn't be changing their OS very often, and when they do, it would be major jumps (e.g. RHEL6 to RHEL7), so they'd be anticipating changes like this.
--
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.
next prev parent reply other threads:[~2013-05-06 13:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-03 19:11 Clarification of labeled IPsec checks Christopher J. PeBenito
2013-05-03 19:20 ` Paul Moore
2013-05-06 13:50 ` Christopher J. PeBenito [this message]
2013-05-07 13:05 ` Paul Moore
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=5187B50C.4020106@tresys.com \
--to=cpebenito@tresys.com \
--cc=pmoore@redhat.com \
--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.