All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel J Walsh <dwalsh@redhat.com>
To: Karl MacMillan <kmacmillan@mentalrootkit.com>
Cc: SE Linux <selinux@tycho.nsa.gov>
Subject: Re: Shell redirection and denials
Date: Tue, 09 Oct 2007 10:23:51 -0400	[thread overview]
Message-ID: <470B8EF7.5030102@redhat.com> (raw)
In-Reply-To: <1191870537.14569.23.camel@localhost.localdomain>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Karl MacMillan wrote:
> One of Dan's constant sources of avcs is something like:
> 
> /usr/sbin/my_confined_app > some_file
> 
> Because the file is created by the shell, opened, and the FD handed to
> the application avcs can occur on read and write.
> 
> Getting rid of these via policy is next to impossible - the destination
> file type is usually governed by the directory and we don't actually
> want to allow that access directly to the confined application. I'd like
> to see if there is some other way to get rid of these denials.
> 
> I see two possible solutions:
> 
> 1) Make the shell create and pass a descriptor to a pipe to the
> application - the shell itself would read / write to the file. This
> seems, to me, to more accurately reflect how we want to enforce the
> permissions.
> 
> 2) Allow applications to confer access by passing the file descriptor
> (more like capabilities). This more closely matches how Unix actually
> works and, of course, is a huge source of vulnerabilities. Allowing this
> type of scheme just for shells might not be that bad.
> 
> Has either of these been investigated? 1 seems pretty simple - is there
> something I'm missing here (perhaps the redirection should outlast the
> shell lifetime?).
> 
> Karl
> 

The biggest source of AVC's that I see reported is the handing off of
open file descriptors.  Mainly terminal descriptors.  Any app that
redirects STDIN/STDERR to a random file location, by opening a file and
handing the descriptor, generates avc messages that cause unexpected
behavior.

service XYZ start >> /tmp/my.log

Will almost always cause avc's and worse no output to my.log.  This
causes sysadmins to go nuts, and it makes no sense to them.

Applications, like rpm, hal, udev, system-config-*, testing tools, any
tools that restarts an init script all do this kind of thing.  As we
move to additional confined user domains and define additional terminal
types this problem proliferates.

The problem is SELinux is preventing READ/WRITE, and does not even look
at Open.  I understand that from a security lock down point of view this
is a big security consideration.  But I believe most sysadmins would
want to prevent their locked down domain to OPEN files in random
location, but if the domains, are handed an OPEN file descriptor from
another process then they should be allowed to READ/WRITE to that OPEN
Descriptor no matter where it is.

So a confined domain could talk to the terminals that were handed to it
but could not open random other terminals.

This is by far the biggest source of dontaudit's allowed in the policy
sources and ends up preventing us from seeing real potential
subversions, where a confined app actually tries to open and talk to
random terminals or the console.



Another side effect I often see is that apps tend to do a getcwd when
they start.  I do not know if this is a standard C/libc activity the way
apps are coded or maybe something about the way bash is coded, but this
ends up generating lots of  AVC messages.

If you look at the confined domains, almost all of them have a
dontaudit $1 sysadm_home_dir_t:dir search_dir_perms;

Because any sysadm who "su -" ends up in the /root homedir and if he
does a service APP restart.  That APP ends up generating and AVC.

I happened to be sitting in a unconfined_tmp_t directory today, when I
started vpnc and boom, setroubleshoot is telling me vpnc_t tried to read
unconfined_tmp_t.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iD8DBQFHC473rlYvE4MpobMRAq2sAKCl5+icAs3Yz42TeEOJrYdmChsZVQCg3tZi
o3ppR6kJaECaGU+sIVifvCE=
=PNLZ
-----END PGP SIGNATURE-----

--
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:[~2007-10-09 16:05 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-08 19:08 Shell redirection and denials Karl MacMillan
2007-10-09 14:23 ` Daniel J Walsh [this message]
2007-10-09 16:55   ` Stephen Smalley
2007-10-10  7:10     ` Kroum Antov
2007-10-10 12:00       ` Stephen Smalley
2007-10-10 16:04         ` Daniel J Walsh
2007-10-10 16:18           ` Stephen Smalley
2007-10-09 14:37 ` Stephen Smalley
2007-10-09 15:04   ` Karl MacMillan
2007-10-09 17:17     ` Christopher J. PeBenito

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=470B8EF7.5030102@redhat.com \
    --to=dwalsh@redhat.com \
    --cc=kmacmillan@mentalrootkit.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.