All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel J Walsh <dwalsh@redhat.com>
To: Stephen Smalley <sds@tycho.nsa.gov>
Cc: ivg2@cornell.edu, selinux@tycho.nsa.gov
Subject: Re: Execmem boolean
Date: Wed, 29 Jun 2005 13:33:36 -0400	[thread overview]
Message-ID: <42C2DB70.2070204@redhat.com> (raw)
In-Reply-To: <1119883756.32316.74.camel@moss-spartans.epoch.ncsc.mil>

Stephen Smalley wrote:

>On Sun, 2005-06-26 at 17:42 -0400, Ivan Gyurdiev wrote:
>  
>
>>Something needs to be done about the allow_execmem boolean - 
>>the default strict policy configuration on Fedora disables it,
>>which causes X not to work out of the box under strict policy. 
>>I think this is highly undesirable.
>>
>>Should allow_execmem be enabled by default?
>>
>>I know Stephen was asking about more fine-grained control
>>over allow_execmem and allow_execmod.
>>
>>How can we combine those two requirements to come up with
>>a better solution?
>>
>>Perhaps have allow_execmem turned on by default,
>>and then and-ed with per application booleans, we can
>>turn it off for selected applications?
>>
>>The opposite approach (off by default, turn on
>>per application) doesn't work quite so well, beause
>>then you have per application booleans not respecting
>>the value of a global allow_execmem...
>>
>>Or, we could get rid of allow_execmem completely, and
>>have only per application booleans...
>>    
>>
>
>Yes, I'm not sure how much value the global allow_execmem/allow_execmod
>booleans provide.  In particular, since execmod should be limited to
>specific file types (although the targeted policy presently violates
>this), it isn't clear that you need a boolean there at all.  I'd say
>that only execmod to types other than texrel_shlib_t truly require a
>boolean.  execmem for the user domains (and all the associated programs
>that run in them) is more problematic.
>
>  
>
Problem is with third party apps,  They don't get labeled texrel_shlib_t 
and therefore do not work.

allow execmod allows them to work with the shlib_t label.  We can change 
this to a targeted vs strict.

>Note also that the execstack/execheap permission checks are now upstream
>and should soon be in the FC devel kernel, so they will also begin to
>have an impact.  execmem is the more general control (make any anonymous
>mapping executable); execstack is a specific case (make the primary
>process stack executable).  Programs that need to generate code at
>runtime will require execmem but should not require execstack.  Programs
>that require an executable stack will require both permissions.  No
>program should legitimately require execheap, but it will take time to
>fix them, starting with X module loader.
>  
>
Did these get added as default allow?  If policy is 19 or less?  Deny if 
policy > 19?

I don't think we should add any more access checks without bumping the 
policy version and putting
that logic in.



>Also note that the legacy binary rules in the current policy are partly
>obsoleted by the checkreqprot change that went into 2.6.12, so that
>unless you are overriding the default checkreqprot value, SELinux is
>only checking the protection requested by the application, so read
>requests should no longer show up as read|execute to SELinux.  This
>should eliminate the need for allowing execute to various file types
>that are only mapped with read access by the application itself.
>
>  
>


-- 



--
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:[~2005-06-29 17:33 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-26 21:42 Execmem boolean Ivan Gyurdiev
2005-06-27 11:59 ` Ivan Gyurdiev
2005-06-27 14:49 ` Stephen Smalley
2005-06-29 17:33   ` Daniel J Walsh [this message]
2005-06-29 18:49     ` Stephen Smalley
2005-06-29 18:54       ` Ivan Gyurdiev
2005-06-29 19:00         ` Stephen Smalley
2005-06-29 19:37         ` Daniel J Walsh
2005-06-29 19:58           ` Stephen Smalley
2005-06-29 20:07           ` Ivan Gyurdiev

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=42C2DB70.2070204@redhat.com \
    --to=dwalsh@redhat.com \
    --cc=ivg2@cornell.edu \
    --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.