All of lore.kernel.org
 help / color / mirror / Atom feed
From: Colin Walters <walters@verbum.org>
To: Daniel J Walsh <dwalsh@redhat.com>
Cc: Stephen Smalley <sds@tycho.nsa.gov>, SELinux <selinux@tycho.nsa.gov>
Subject: Re: Force avc_has_perm to return success if enforcing == 0;
Date: Thu, 16 Feb 2012 10:18:08 -0500	[thread overview]
Message-ID: <1329405489.2867.8.camel@lenny> (raw)
In-Reply-To: <4F3AD084.5060304@redhat.com>

On Tue, 2012-02-14 at 16:22 -0500, Daniel J Walsh wrote:
> I would like to patch libselinux to always return 0 on avc_has_perm if
> the machine is in permissive mode.
> 
> This will allow Userspace Object Managers to work even if the system
> is totally mislabeled and processes as running with bad context.
> Currently if a program like dbus asks with a bad process label it can
> get denials even in permissive mode.

The patch seems like it could at least use a comment.

For reference the previous discussion was here:

http://marc.info/?l=selinux&m=123973525511476&w=2

I'm not opposed to the proposed patch, but ultimately while it will
make their system not crash when in permissive, it also won't help
much in getting people towards an enforcing system.  It seems to me
we need to improve:

1) The notification that the system requires a relabel.  If terminals
   were HTML, we'd be using <blink>
2) The process of relabeling.
   I think fixfiles defaulting to "all files on all mounted filesystems"
   is kind of wrong.  We should only relabel things installed by RPM
   first, which should get you back to a clean slate for the system,
   then you can look at relabeling your custom web content, etc.



--
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:[~2012-02-16 15:18 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-14 21:22 Force avc_has_perm to return success if enforcing == 0; Daniel J Walsh
2012-02-16 14:25 ` Stephen Smalley
2012-02-16 14:37   ` Daniel J Walsh
2012-02-21 20:37     ` Stephen Smalley
2012-02-16 15:18 ` Colin Walters [this message]
     [not found] <CAPzO=Nw+T8_QkzHPboQ-s399hmdMF0jb0_3ipSrSyqpHztfOfg@mail.gmail.com>
2012-02-15 15:25 ` 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=1329405489.2867.8.camel@lenny \
    --to=walters@verbum.org \
    --cc=dwalsh@redhat.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.