Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754071AbcCHOnX (ORCPT ); Tue, 8 Mar 2016 09:43:23 -0500 Received: from mx1.redhat.com ([209.132.183.28]:36211 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751531AbcCHOnO (ORCPT ); Tue, 8 Mar 2016 09:43:14 -0500 Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 3798903 From: David Howells In-Reply-To: <1457447480.5321.115.camel@linux.vnet.ibm.com> References: <1457447480.5321.115.camel@linux.vnet.ibm.com> <1457402735.5321.14.camel@linux.vnet.ibm.com> <20160304150022.17121.34501.stgit@warthog.procyon.org.uk> <20160304150142.17121.56666.stgit@warthog.procyon.org.uk> <30699.1457442839@warthog.procyon.org.uk> To: Mimi Zohar Cc: dhowells@redhat.com, linux-security-module@vger.kernel.org, keyrings@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 11/12] certs: Add a secondary system keyring that can be added to dynamically [ver #2] MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <448.1457448192.1@warthog.procyon.org.uk> Date: Tue, 08 Mar 2016 14:43:12 +0000 Message-ID: <449.1457448192@warthog.procyon.org.uk> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 921 Lines: 23 Mimi Zohar wrote: > Would then restrict_link_to_system_trusted imply both the builtin and > secondary keyrings or just the builtin keyrings? Both, if available; just builtin if the secondary is not available. restrict_link_by_builtin_trusted() does only the builtin. > Changing the system keyring name to builtin keys, without changing the > corresponding restrict_link name, obfuscates what is really happening. You can still look at the code, it's not as if it's particularly hidden. The problem boils down to a difficulty in concocting a name that describes a complex situation that may change depending on the configuration. I can make it "restrict_link_by_any_system_trusted" if you'd prefer. That's why I want "system trusted keyrings" to refer to the builtin and the secondary - *and* an extra UEFI keyring if we grow one of those. It's a collection of related keyrings. David