From: Andy Lutomirski <luto@stanford.edu>
To: Michael Glasgow <glasgow@beer.net>, linux-kernel@vger.kernel.org
Subject: Re: posix capabilities inheritance
Date: Tue, 21 Oct 2003 11:24:17 -0700 [thread overview]
Message-ID: <3F9579D1.6050108@stanford.edu> (raw)
In-Reply-To: <fa.hehull9.10mmngt@ifi.uio.no>
I agree with these problems, but I think the proper fix is complicated. AFAICT,
POSIX capability evolution, as specified by whatever draft it was, is broken, so
the hacks in prepare_binprm (cap_bprm_set_security in 2.6) are needed to avoid
security problems. Aside from the fact that non-inheritable-by-default makes
little sense (and requires root to get capabilities re-added from the file
_permitted_ set), POSIX cap evolution has some other problems:
1. Can a process have capabilities in its inheritable set and not in its
permitted set? POSIX allows such processes to be created (pI = pP = full, then
execute (fI = 0, fP = 0). Nevertheless, its pP evolution rule assumes that this
never happens (pI capabilities can reappear).
2. If a process has pE < pP (i.e. some caps disabled, e.g. uid=0, euid!=0), and
exec's fE=full, then its capabilities get re-enabled. This seems like a pretty
serious breakage of userspace.
I have a possible fix to these issues at
<http://www.stanford.edu/~luto/linux-fscap/> -- CONFIG_FS_CAPS (in
cap-1-fscap.patch) changes the capability evolution rules slightly to make them
(IMHO) both safe and sensible, as well as removing the ugly hackage in
set_security. This will allow your capabilities to persist through exec(). The
second patch (cap-2-ext3.patch) adds file capabilities to ext3. Both are tested
on 2.6.0-test6, and they apply with fuzz and compile (but not tested yet) on
-test8. (These are different from my last attempt at this -- they are much
closer to POSIX semantics.)
Unfortunately, I think it's too late to include these in 2.6.0.
Andy
next parent reply other threads:[~2003-10-21 18:24 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <fa.hehull9.10mmngt@ifi.uio.no>
2003-10-21 18:24 ` Andy Lutomirski [this message]
[not found] <fa.f26d55g.1qgijbi@ifi.uio.no>
[not found] ` <fa.hq0dft9.9i0obd@ifi.uio.no>
2003-10-24 21:24 ` posix capabilities inheritance Andy Lutomirski
[not found] <fa.n4rmmgg.2423pm@ifi.uio.no>
[not found] ` <fa.l1oevhb.1s5k583@ifi.uio.no>
2003-10-24 8:44 ` Andy Lutomirski
2003-10-24 12:41 ` Theodore Ts'o
2003-10-24 16:44 ` Andy Lutomirski
2003-10-24 20:58 ` David Wagner
[not found] <fa.f4bs2b4.fhub0m@ifi.uio.no>
2003-10-23 22:05 ` Michael Glasgow
2003-10-23 22:59 ` Theodore Ts'o
2003-10-24 1:36 ` Ernie Petrides
2003-10-24 2:19 ` Bernd Eckenfels
2003-10-24 5:10 ` Ernie Petrides
2003-10-25 19:51 ` Pavel Machek
2003-10-23 1:41 Albert Cahalan
[not found] <fa.f9mv0tb.27sf3j@ifi.uio.no>
2003-10-23 1:36 ` Michael Glasgow
2003-10-23 1:57 ` Andy Lutomirski
[not found] <fa.f36f4t9.1rg8j3v@ifi.uio.no>
2003-10-22 7:11 ` Michael Glasgow
2003-10-22 19:40 ` Andy Lutomirski
-- strict thread matches above, loose matches on Subject: below --
2003-10-21 11:26 Michael Glasgow
2003-10-23 16:46 ` Theodore Ts'o
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=3F9579D1.6050108@stanford.edu \
--to=luto@stanford.edu \
--cc=glasgow@beer.net \
--cc=linux-kernel@vger.kernel.org \
/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.