MIME-Version: 1.0
In-Reply-To: <1441165462.17898.94.camel@linux.vnet.ibm.com>
References: <1440462367.2737.4.camel@linux.vnet.ibm.com>
	<CALCETrXWBBdOKz-fSdM7YVu_sWQbA3YsHPeZAkRmtj+eawqZGQ@mail.gmail.com>
	<1440464705.2737.36.camel@linux.vnet.ibm.com>
	<14540.1440599584@warthog.procyon.org.uk>
	<31228.1440671938@warthog.procyon.org.uk>
	<36ddb60c1d22756234392a2d065a02cb.squirrel@twosheds.infradead.org>
	<20150827212907.GF8051@wotan.suse.de>
	<1440719673.2118.84.camel@linux.vnet.ibm.com>
	<20150829021659.GN8051@wotan.suse.de>
	<1441030735.2647.70.camel@linux.vnet.ibm.com>
	<20150901234305.GU8051@wotan.suse.de>
	<CAGXu5j+_yHc1wbubzw4h=QFM=uyn1pCraGYCWS6LvgtS3-A+Qw@mail.gmail.com>
	<1441165462.17898.94.camel@linux.vnet.ibm.com>
Date: Wed, 2 Sep 2015 08:28:03 -0700
Message-ID: <CAGXu5jJ7XGzYD=HX13SavVXSqZ_DvOKU2udym91tM4RFknapZw@mail.gmail.com> (sfid-20150902_172811_371542_D3A21058)
Subject: Re: Linux Firmware Signing
From: Kees Cook <keescook@chromium.org>
To: Mimi Zohar <zohar@linux.vnet.ibm.com>
Cc: "Luis R. Rodriguez" <mcgrof@suse.com>,
	David Woodhouse <dwmw2@infradead.org>,
	David Howells <dhowells@redhat.com>,
	Andy Lutomirski <luto@amacapital.net>,
	"Roberts, William C" <william.c.roberts@intel.com>,
	"linux-security-module@vger.kernel.org"
	<linux-security-module@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	"james.l.morris@oracle.com" <james.l.morris@oracle.com>,
	"serge@hallyn.com" <serge@hallyn.com>,
	Vitaly Kuznetsov <vkuznets@redhat.com>,
	Paul Moore <paul@paul-moore.com>,
	Eric Paris <eparis@parisplace.org>,
	SE Linux <selinux@tycho.nsa.gov>,
	Stephen Smalley <sds@tycho.nsa.gov>,
	"Schaufler, Casey" <casey.schaufler@intel.com>,
	"Luis R. Rodriguez" <mcgrof@do-not-panic.com>,
	Dmitry Kasatkin <dmitry.kasatkin@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Peter Jones <pjones@redhat.com>, Takashi Iwai <tiwai@suse.de>,
	Ming Lei <ming.lei@canonical.com>, Joey Lee <jlee@suse.de>,
	=?UTF-8?B?Vm9qdMSbY2ggUGF2bMOtaw==?= <vojtech@suse.com>,
	Kyle McMartin <kyle@kernel.org>,
	Seth Forshee <seth.forshee@canonical.com>,
	Matthew Garrett <mjg59@srcf.ucam.org>,
	Johannes Berg <johannes@sipsolutions.net>,
	Julia Lawall <julia.lawall@lip6.fr>,
	Jay Schulist <jschlst@samba.org>,
	Daniel Borkmann <dborkman@redhat.com>,
	Alexei Starovoitov <ast@plumgrid.com>
Content-Type: text/plain; charset=UTF-8
Sender: linux-wireless-owner@vger.kernel.org

On Tue, Sep 1, 2015 at 8:44 PM, Mimi Zohar <zohar@linux.vnet.ibm.com> wrote:
> On Tue, 2015-09-01 at 20:08 -0700, Kees Cook wrote:
>> On Tue, Sep 1, 2015 at 4:43 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>> > On Mon, Aug 31, 2015 at 10:18:55AM -0400, Mimi Zohar wrote:
>> >> > > eBPF/seccomp
>> >
>> > OK I knew nothing about this but I just looked into it, here are my notes:
>> >
>> >   * old BPF - how far do we want to go? This goes so far as to parsing
>> >     user passed void __user *arg data through ioctls which typically
>> >     gets copy_from_user()'d and eventually gets BPF_PROG_RUN().
>> >
>> >   * eBPF:
>> >                              seccomp() & prctl_set_seccomp()
>> >                                         |
>> >                                         V
>> >                              do_seccomp()
>> >                                         |
>> >                                         V
>> >                              seccomp_set_mode_filter()
>> >                                         |
>> >                                         V
>> >                              seccomp_prepare_user_filter()
>> >                                         |
>> >                                         V
>> >         bpf_prog_create_from_user() (seccomp) \
>> >         bpf_prog_create()                      > bpf_prepare_filter()
>> >         sk_attach_filter()                    /
>> >
>> >     All approaches come from user passed data, nothing fd based.
>> >
>> >     For both old BPF and eBPF then:
>> >
>> >     If we wanted to be paranoid I suppose the Machine Owner Key (MOK)
>> >     Paul had mentioned up could be used to vet for passed filters, or
>> >     a new interface to enable fd based filters. This really would limit
>> >     the dynamic nature of these features though.
>> >
>> >     eBPF / secccomp would not be the only place in the kernel that would have
>> >     issues with user passed data, we have tons of places the same applies so
>> >     implicating the old BPF / eBPF / seccomp approaches can easily implicate
>> >     many other areas of the kernel, that's pretty huge but from the looks of
>> >     it below you seem to enable that to be a possibility for us to consider.
>>
>> At the time (LSS 2014?) I argued that seccomp policies come from
>> binaries, which are already being measured. And that policies only
>> further restrict a process, so there seems to be to be little risk in
>> continuing to leave them unmeasured.
>
> What do you mean by "measured"?  Who is doing the measurement?  Could
> someone detect a change in measurement?

I meant from the perspective of IMA. The binary would have already
been evaluated when it executed, and it's what's installing the
seccomp filter. And since seccomp filters can only reduce privilege,
it seems like they're not worth getting processed by IMA. But I might
not understand the requirements! :)

-Kees

-- 
Kees Cook
Chrome OS Security