Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756666Ab2HTOXF (ORCPT ); Mon, 20 Aug 2012 10:23:05 -0400 Received: from mail-qc0-f174.google.com ([209.85.216.174]:34284 "EHLO mail-qc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756651Ab2HTOXB (ORCPT ); Mon, 20 Aug 2012 10:23:01 -0400 MIME-Version: 1.0 In-Reply-To: <1345468396.2355.27.camel@falcor> References: <2114492cd221edc44622e528d66feeed342d2d34.1345055639.git.dmitry.kasatkin@intel.com> <1345164802.2433.21.camel@falcor.watson.ibm.com> <1345223322.2257.75.camel@falcor> <1345424724.2384.15.camel@falcor> <1345468396.2355.27.camel@falcor> Date: Mon, 20 Aug 2012 10:23:00 -0400 Message-ID: Subject: Re: [RFC v2 7/7] modsig: build rules and scripts to generate keys and sign modules From: Josh Boyer To: Mimi Zohar Cc: "Kasatkin, Dmitry" , jmorris@namei.org, rusty@rustcorp.com.au, dhowells@redhat.com, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5392 Lines: 105 On Mon, Aug 20, 2012 at 9:13 AM, Mimi Zohar wrote: > On Mon, 2012-08-20 at 08:32 -0400, Josh Boyer wrote: >> On Sun, Aug 19, 2012 at 9:05 PM, Mimi Zohar wrote: >> > On Fri, 2012-08-17 at 13:44 -0400, Josh Boyer wrote: >> >> On Fri, Aug 17, 2012 at 1:08 PM, Mimi Zohar wrote: >> >> >> I don't see a need for the kernel make system to ever delete a key. >> >> >> If one doesn't exist, it should create one if the config options are >> >> >> set and leave it alone entirely after that. If one exists already, >> >> >> then it should leave it alone as it already does. >> >> > >> >> > Ok. Other than generating a key the first time, the normal development >> >> > build process now uses the same key, never requiring the developer to do >> >> > anything additional. The developer controls the frequency the keys are >> >> > created/deleted. I wonder how often that will be ... >> >> >> >> For a developer, probably not often. Though in reality, the usage of >> >> this by a single developer seems fair small. A much wider, already >> >> existing usage is distribution kernel signing. >> > >> > This seems to be the real issue. Do developers need to sign their own >> > builds? Nobody is forcing the developer to enable signed modules. If, >> > however, the developer decides to enforce signed modules, leaving the >> > private key lying around kind of defeats the purpose. >> >> Right. The issue is that you're making the kernel buildsystem do key >> management policy instead of trusting the developers using the keys >> mechanism to know what they're doing. >> >> >> For a distribution kernel there are normally two cases: >> >> >> >> 1) Existing permanent company/distro key. This is already handled. >> >> 2) per-kernel-build key (equivalent to your ephemeral key). This would >> >> be handled fine if the key was generated if it didn't already exist and >> >> left alone. The distro build system would remove it when it cleaned up >> >> the buildroot. >> > >> > Ok, I think we agree here, with the normal 'make', 'make modules_install >> > install', using an existing key or creating a new key if needed. >> > >> >> >> If you really want to enforce ephemeral keys in the make system, then >> >> >> doing it via 'make clean' or 'make distclean' would make more sense to >> >> >> me. But I personally think key management is something the developers >> >> >> or distros should be handling on their own. Creating a key for them is >> >> >> a convenience so it's worthwhile, but removing it should be done by >> >> >> them. >> >> > >> >> > Sorry, I disagree. Without the signed_modules_install target, the >> >> > developer would need to do each step manually - create new key, sign >> >> > modules, remove private key, and embed the new public key in the >> >> > bzImage. >> >> >> >> Not if the buildsystem just made the key for them at 'make' time. I'm >> >> still missing why it isn't done until 'modules_install' time. Seems >> >> pretty sub-optimal. >> > >> > Using a private key that has been lying around for an unknown period of >> > time, kind of defeats the purpose. >> >> Why is that? Again, you're enforcing policy instead of leaving it up >> to the developers. In the existing key case, you have no idea how long >> that key has been sitting around either, so I really see a disconnect >> between "existing company/private key" and "no key, create one for them >> during build time" cases. > > Either the company build process, creates and cleans up after the build, > removing the private key, as you described above, or the "private key" > is held securely using an existing signing process (or some combination > of the two). > > In the "no key, create one for them during build time", there is no > process in place cleaning up, other than the suggested new make target. You could have 'make clean/distclean' remove the key if you really wanted. >> >> If >> >> the config options are set, just have 'make modules_install' do it and >> >> create a key if one doesn't exist (or better yet, have 'make' do it). >> > >> > If creating the key is deferred to 'make modules_install', for the >> > reason given above or any other reason, the public key wasn't available >> > at the time the bzImage was created. 'make modules_install' would now >> > need to rebuild the bzImage. >> >> Right, which is an utter pain and backwards. That's why I suggested it >> be done at 'make' time. > > Agreed, no doubt about it being a pain or backwards, but the solution > isn't to deny the problem, but to address it. I'm open to > suggestions. :) I've made multiple suggestions. I have no more. Either you agree with them or your don't. Our fundamental disagreement is the management of automatically created keys. You want them removed ASAP, I suggest that people/companies/distributions using that mechanism should be aware of what they're doing and be responsible for removing keys when they see fit. Aside from making the behavior dependent on perhaps some kind of "expert" config setting, I don't see a way past that difference. josh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/