From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932255AbdJUTIO convert rfc822-to-8bit (ORCPT ); Sat, 21 Oct 2017 15:08:14 -0400 Received: from smtp-sh.infomaniak.ch ([128.65.195.4]:36744 "EHLO smtp-sh.infomaniak.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932159AbdJUTIL (ORCPT ); Sat, 21 Oct 2017 15:08:11 -0400 From: nicolas@belouin.fr Message-Id: <201710211904.v9LJ4o1X046943@smtp5.infomaniak.ch> Date: Sat, 21 Oct 2017 21:04:48 +0200 User-Agent: K-9 Mail for Android In-Reply-To: References: <20171021134558.21195-1-nicolas@belouin.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Subject: Re: [kernel-hardening] [RFC PATCH 1/2] security, capabilities: create CAP_TRUSTED To: Casey Schaufler , Jan Kara , "Theodore Ts'o" , Andreas Dilger , Jaegeuk Kim , Chao Yu , David Woodhouse , Dave Kleikamp , Mark Fasheh , Joel Becker , Miklos Szeredi , Phillip Lougher , Richard Weinberger , Artem Bityutskiy , Adrian Hunter , Alexander Viro , Serge Hallyn , Paul Moore , Stephen Smalley , Eric Paris , James.Morris@smtp5.infomaniak.ch X-Antivirus: Dr.Web (R) for Unix mail servers drweb plugin ver.6.0.2.8 X-Antivirus-Code: 0x100000 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ,linux-ext4@vger.kernel.org,linux-kernel@vger.kernel.org,linux-f2fs-devel@lists.sourceforge.net,linux-fsdevel@vger.kernel.org,linux-mtd@lists.infradead.org,jfs-discussion@lists.sourceforge.net,ocfs2-devel@oss.oracle.com,linux-unionfs@vger.kernel.org,reiserfs-devel@vger.kernel.org,linux-security-module@vger.kernel.org,selinux@tycho.nsa.gov,linux-api@vger.kernel.org,kernel-hardening@lists.openwall.com From: Nicolas Belouin Message-ID: On October 21, 2017 7:25:21 PM GMT+02:00, Casey Schaufler wrote: >On 10/21/2017 6:45 AM, Nicolas Belouin wrote: >> with CAP_SYS_ADMIN being bloated, the usefulness of using it to >> flag a process to be entrusted for e.g reading and writing trusted >> xattr is near zero. >> CAP_TRUSTED aims to provide userland with a way to mark a process as >> entrusted to do specific (not specially admin-centered) actions. It >> would for example allow a process to red/write the trusted xattrs. > >Please explain how this is different from CAP_MAC_ADMIN in >any existing use case. If it is significantly different, how >would the two interact? >>From my point of view, CAP_MAC_ADMIN allows one to read/write security xattrs, those are meant to describe security policies. As far as I know of, trusted xattrs are intended for a privileged process to read or write arbitrary data. I don't have any real world example in mind that use trusted xattrs, but I'll try to find one. > >> Signed-off-by: Nicolas Belouin >> --- >> include/uapi/linux/capability.h | 6 +++++- >> security/selinux/include/classmap.h | 5 +++-- >> 2 files changed, 8 insertions(+), 3 deletions(-) >> >> diff --git a/include/uapi/linux/capability.h >b/include/uapi/linux/capability.h >> index ce230aa6d928..27e457b93c84 100644 >> --- a/include/uapi/linux/capability.h >> +++ b/include/uapi/linux/capability.h >> @@ -369,7 +369,11 @@ struct vfs_ns_cap_data { >> >> #define CAP_SYS_MOUNT 38 >> >> -#define CAP_LAST_CAP CAP_SYS_MOUNT >> +/* Allow read/write trusted xattr */ >> + >> +#define CAP_TRUSTED 39 >> + >> +#define CAP_LAST_CAP CAP_TRUSTED >> >> #define cap_valid(x) ((x) >= 0 && (x) <= CAP_LAST_CAP) >> >> diff --git a/security/selinux/include/classmap.h >b/security/selinux/include/classmap.h >> index a873dce97fd5..f5dc8e109f5a 100644 >> --- a/security/selinux/include/classmap.h >> +++ b/security/selinux/include/classmap.h >> @@ -24,9 +24,10 @@ >> "audit_control", "setfcap" >> >> #define COMMON_CAP2_PERMS "mac_override", "mac_admin", "syslog", \ >> - "wake_alarm", "block_suspend", "audit_read", "sys_mount" >> + "wake_alarm", "block_suspend", "audit_read", "sys_mount", \ >> + "trusted" >> >> -#if CAP_LAST_CAP > CAP_SYS_MOUNT >> +#if CAP_LAST_CAP > CAP_TRUSTED >> #error New capability defined, please update COMMON_CAP2_PERMS. >> #endif >> Nicolas From mboxrd@z Thu Jan 1 00:00:00 1970 From: nicolas@belouin.fr (nicolas at belouin.fr) Date: Sat, 21 Oct 2017 21:04:48 +0200 Subject: [kernel-hardening] [RFC PATCH 1/2] security, capabilities: create CAP_TRUSTED In-Reply-To: References: <20171021134558.21195-1-nicolas@belouin.fr> Message-ID: <201710211904.v9LJ4o1X046943@smtp5.infomaniak.ch> To: linux-security-module@vger.kernel.org List-Id: linux-security-module.vger.kernel.org ,linux-ext4 at vger.kernel.org,linux-kernel at vger.kernel.org,linux-f2fs-devel at lists.sourceforge.net,linux-fsdevel at vger.kernel.org,linux-mtd at lists.infradead.org,jfs-discussion at lists.sourceforge.net,ocfs2-devel at oss.oracle.com,linux-unionfs at vger.kernel.org,reiserfs-devel at vger.kernel.org,linux-security-module at vger.kernel.org,selinux at tycho.nsa.gov,linux-api at vger.kernel.org,kernel-hardening at lists.openwall.com From: Nicolas Belouin Message-ID: On October 21, 2017 7:25:21 PM GMT+02:00, Casey Schaufler wrote: >On 10/21/2017 6:45 AM, Nicolas Belouin wrote: >> with CAP_SYS_ADMIN being bloated, the usefulness of using it to >> flag a process to be entrusted for e.g reading and writing trusted >> xattr is near zero. >> CAP_TRUSTED aims to provide userland with a way to mark a process as >> entrusted to do specific (not specially admin-centered) actions. It >> would for example allow a process to red/write the trusted xattrs. > >Please explain how this is different from CAP_MAC_ADMIN in >any existing use case. If it is significantly different, how >would the two interact? >>From my point of view, CAP_MAC_ADMIN allows one to read/write security xattrs, those are meant to describe security policies. As far as I know of, trusted xattrs are intended for a privileged process to read or write arbitrary data. I don't have any real world example in mind that use trusted xattrs, but I'll try to find one. > >> Signed-off-by: Nicolas Belouin >> --- >> include/uapi/linux/capability.h | 6 +++++- >> security/selinux/include/classmap.h | 5 +++-- >> 2 files changed, 8 insertions(+), 3 deletions(-) >> >> diff --git a/include/uapi/linux/capability.h >b/include/uapi/linux/capability.h >> index ce230aa6d928..27e457b93c84 100644 >> --- a/include/uapi/linux/capability.h >> +++ b/include/uapi/linux/capability.h >> @@ -369,7 +369,11 @@ struct vfs_ns_cap_data { >> >> #define CAP_SYS_MOUNT 38 >> >> -#define CAP_LAST_CAP CAP_SYS_MOUNT >> +/* Allow read/write trusted xattr */ >> + >> +#define CAP_TRUSTED 39 >> + >> +#define CAP_LAST_CAP CAP_TRUSTED >> >> #define cap_valid(x) ((x) >= 0 && (x) <= CAP_LAST_CAP) >> >> diff --git a/security/selinux/include/classmap.h >b/security/selinux/include/classmap.h >> index a873dce97fd5..f5dc8e109f5a 100644 >> --- a/security/selinux/include/classmap.h >> +++ b/security/selinux/include/classmap.h >> @@ -24,9 +24,10 @@ >> "audit_control", "setfcap" >> >> #define COMMON_CAP2_PERMS "mac_override", "mac_admin", "syslog", \ >> - "wake_alarm", "block_suspend", "audit_read", "sys_mount" >> + "wake_alarm", "block_suspend", "audit_read", "sys_mount", \ >> + "trusted" >> >> -#if CAP_LAST_CAP > CAP_SYS_MOUNT >> +#if CAP_LAST_CAP > CAP_TRUSTED >> #error New capability defined, please update COMMON_CAP2_PERMS. >> #endif >> Nicolas -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo at vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-sh.infomaniak.ch ([128.65.195.4]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1e5z6R-0007pc-FI for linux-mtd@lists.infradead.org; Sat, 21 Oct 2017 19:06:38 +0000 From: nicolas@belouin.fr Message-Id: <201710211904.v9LJ4o1X046943@smtp5.infomaniak.ch> Date: Sat, 21 Oct 2017 21:04:48 +0200 In-Reply-To: References: <20171021134558.21195-1-nicolas@belouin.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [kernel-hardening] [RFC PATCH 1/2] security, capabilities: create CAP_TRUSTED To: Casey Schaufler , Jan Kara , "Theodore Ts'o" , Andreas Dilger , Jaegeuk Kim , Chao Yu , David Woodhouse , Dave Kleikamp , Mark Fasheh , Joel Becker , Miklos Szeredi , Phillip Lougher , Richard Weinberger , Artem Bityutskiy , Adrian Hunter , Alexander Viro , Serge Hallyn , Paul Moore , Stephen Smalley , Eric Paris , James.Morris@smtp5.infomaniak.ch List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , ,linux-ext4@vger.kernel.org,linux-kernel@vger.kernel.org,linux-f2fs-devel@lists.sourceforge.net,linux-fsdevel@vger.kernel.org,linux-mtd@lists.infradead.org,jfs-discussion@lists.sourceforge.net,ocfs2-devel@oss.oracle.com,linux-unionfs@vger.kernel.org,reiserfs-devel@vger.kernel.org,linux-security-module@vger.kernel.org,selinux@tycho.nsa.gov,linux-api@vger.kernel.org,kernel-hardening@lists.openwall.com From: Nicolas Belouin Message-ID: On October 21, 2017 7:25:21 PM GMT+02:00, Casey Schaufler wrote: >On 10/21/2017 6:45 AM, Nicolas Belouin wrote: >> with CAP_SYS_ADMIN being bloated, the usefulness of using it to >> flag a process to be entrusted for e=2Eg reading and writing trusted >> xattr is near zero=2E >> CAP_TRUSTED aims to provide userland with a way to mark a process as >> entrusted to do specific (not specially admin-centered) actions=2E It >> would for example allow a process to red/write the trusted xattrs=2E > >Please explain how this is different from CAP_MAC_ADMIN in >any existing use case=2E If it is significantly different, how >would the two interact? From=20my point of view, CAP_MAC_ADMIN allows one to read/write security xat= trs, those are meant to describe security policies=2E As far as I know of, = trusted xattrs are intended for a privileged process to read or write arbit= rary data=2E I don't have any real world example in mind that use trusted x= attrs, but I'll try to find one=2E > >> Signed-off-by: Nicolas Belouin >> --- >> include/uapi/linux/capability=2Eh | 6 +++++- >> security/selinux/include/classmap=2Eh | 5 +++-- >> 2 files changed, 8 insertions(+), 3 deletions(-) >> >> diff --git a/include/uapi/linux/capability=2Eh >b/include/uapi/linux/capability=2Eh >> index ce230aa6d928=2E=2E27e457b93c84 100644 >> --- a/include/uapi/linux/capability=2Eh >> +++ b/include/uapi/linux/capability=2Eh >> @@ -369,7 +369,11 @@ struct vfs_ns_cap_data { >> =20 >> #define CAP_SYS_MOUNT 38 >> =20 >> -#define CAP_LAST_CAP CAP_SYS_MOUNT >> +/* Allow read/write trusted xattr */ >> + >> +#define CAP_TRUSTED 39 >> + >> +#define CAP_LAST_CAP CAP_TRUSTED >> =20 >> #define cap_valid(x) ((x) >=3D 0 && (x) <=3D CAP_LAST_CAP) >> =20 >> diff --git a/security/selinux/include/classmap=2Eh >b/security/selinux/include/classmap=2Eh >> index a873dce97fd5=2E=2Ef5dc8e109f5a 100644 >> --- a/security/selinux/include/classmap=2Eh >> +++ b/security/selinux/include/classmap=2Eh >> @@ -24,9 +24,10 @@ >> "audit_control", "setfcap" >> =20 >> #define COMMON_CAP2_PERMS "mac_override", "mac_admin", "syslog", \ >> - "wake_alarm", "block_suspend", "audit_read", "sys_mount" >> + "wake_alarm", "block_suspend", "audit_read", "sys_mount", \ >> + "trusted" >> =20 >> -#if CAP_LAST_CAP > CAP_SYS_MOUNT >> +#if CAP_LAST_CAP > CAP_TRUSTED >> #error New capability defined, please update COMMON_CAP2_PERMS=2E >> #endif >> =20 Nicolas