All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Caplan <dac@tresys.com>
To: Chris PeBenito <pebenito@gentoo.org>
Cc: Stephen Smalley <sds@epoch.ncsc.mil>,
	Russell Coker <russell@coker.com.au>,
	SE Linux <selinux@tycho.nsa.gov>
Subject: Re: specifying groups of types
Date: Mon, 03 Nov 2003 08:55:07 -0500	[thread overview]
Message-ID: <3FA65E3B.6080008@tresys.com> (raw)
In-Reply-To: <1067794997.5374.42.camel@chris.pebenito.net>

I didn't receive any comments on it.  I'd emphasize again that my 
solution was not a "proper" fix (it leaves open too many 
inconsistencies) and was not intended to be accepted as an official 
patch until it was cleaned up.  If no one comes up with a clean solution 
I'll fix mine when I have the chance.  Right now I'm trying to finish up 
the parsing code for our conditional policy modification among other things.

David

Chris PeBenito wrote:
> Was there any resolution to this?  I think this would be useful for
> checkpolicy to have, but it hasn't been merged (at least on the
> sourceforge cvs).  I don't remember seeing any official response from
> the NSA team about it nor any criticism/improvements of it.
> 
> On Tue, 2003-10-14 at 13:56, David Caplan wrote:
> 
>>Russell,
>>
>>Here's a quick hack that appears to work.  It turns off the type (or 
>>list of types if used on an attribute) when building the bitmap of types 
>>for a rule.  The syntax is to use a '-' in front of a type or attribute 
>>name.
>>
>>allow some_domain { file_type -shadow_t -null_device_t -exec_type}:...
>>
>>The proper way to do this is in the yacc parsing section.  All I did was 
>>allow '-' as the first character of an identifier (policy_scan.l) and 
>>handle the subtraction of the type/attribute in 
>>policy_parse.y:set_types().  The danger is that types (and anything 
>>using the identifier definition) can be declared with '-' as the first 
>>character and cause problems.  The advantage, in theory, is that 
>>wherever a list of types/attributes is processed, the '-' notation can 
>>be used to turn off types.  So, you should also be able to do something 
>>like:
>>
>>allow { auth -crond_t } file_type:...
>>
>>Types/attributes are processed in order, and subsequent allow rules can 
>>also override the subtraction.
>>
>>I'd recommend trying this out and if you find it useful change the parse 
>>rules.  I tested it on some real basic policy, so it may cause other 
>>unintended problems.  I'm throwing it out more as a starting point 
>>rather than something intended to be integrated into checkpolicy.
>>
>>David
>>
>>Stephen Smalley wrote:
>>
>>>On Sat, 2003-10-11 at 00:35, Russell Coker wrote:
>>>
>>>
>>>>Following a discussion on IRC, it occurs to me that it would be handy to have 
>>>>the following in the policy language:
>>>>allow some_domain { file_type !shadow_t }:...
>>>>
>>>>So we can specify everything in file_type except for shadow_t.
>>>
>>>
>>>Yes, although I'm not sure about the notation; might be better to
>>>provide a set difference operator, e.g.
>>>	file_type - shadow_t
>>>
>>>Are you offering to implement this enhancement to checkpolicy?
>>>


-- 
__________________________________

David Caplan     410 290 1411 x105
dac@tresys.com
Tresys Technology, LLC
8840 Stanford Blvd., Suite 2100
Columbia, MD 21045


--
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.

  parent reply	other threads:[~2003-11-03 13:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-11  4:35 specifying groups of types Russell Coker
2003-10-14 12:22 ` Stephen Smalley
2003-10-14 18:56   ` David Caplan
2003-10-16  5:07     ` Chris PeBenito
2003-11-02 17:43     ` Chris PeBenito
2003-11-03 13:53       ` Stephen Smalley
2003-11-03 13:55       ` David Caplan [this message]
2004-01-15 14:31     ` Stephen Smalley
2004-01-16 16:50       ` Karl MacMillan
2004-01-16 18:17         ` Stephen Smalley
2004-01-16 18:36           ` Karl MacMillan
2004-01-16 18:48             ` Stephen Smalley
2004-01-16 21:03           ` Trival relabel Problem Thomas DuBuisson
2004-01-16 21:14             ` 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=3FA65E3B.6080008@tresys.com \
    --to=dac@tresys.com \
    --cc=pebenito@gentoo.org \
    --cc=russell@coker.com.au \
    --cc=sds@epoch.ncsc.mil \
    --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.