From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753562AbcHZOVC (ORCPT ); Fri, 26 Aug 2016 10:21:02 -0400 Received: from mail-ua0-f172.google.com ([209.85.217.172]:35706 "EHLO mail-ua0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753508AbcHZOU5 (ORCPT ); Fri, 26 Aug 2016 10:20:57 -0400 MIME-Version: 1.0 In-Reply-To: <57BF043D.4000300@digikod.net> References: <1472121165-29071-1-git-send-email-mic@digikod.net> <1472121165-29071-10-git-send-email-mic@digikod.net> <57BF043D.4000300@digikod.net> From: Andy Lutomirski Date: Fri, 26 Aug 2016 07:20:35 -0700 Message-ID: Subject: Re: [RFC v2 09/10] landlock: Handle cgroups To: =?UTF-8?B?TWlja2HDq2wgU2FsYcO8bg==?= Cc: LKML , Alexei Starovoitov , Arnd Bergmann , Casey Schaufler , Daniel Borkmann , Daniel Mack , David Drysdale , "David S . Miller" , Elena Reshetova , James Morris , Kees Cook , Paul Moore , Sargun Dhillon , "Serge E . Hallyn" , Will Drewry , Kernel Hardening , Linux API , LSM List , Network Development , "open list:CONTROL GROUP (CGROUP)" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id u7QEOZRD011141 On Thu, Aug 25, 2016 at 7:44 AM, Mickaël Salaün wrote: > > On 25/08/2016 13:09, Andy Lutomirski wrote: >> On Thu, Aug 25, 2016 at 3:32 AM, Mickaël Salaün wrote: >>> Add an eBPF function bpf_landlock_cmp_cgroup_beneath(opt, map, map_op) >>> to compare the current process cgroup with a cgroup handle, The handle >>> can match the current cgroup if it is the same or a child. This allows >>> to make conditional rules according to the current cgroup. >>> >>> A cgroup handle is a map entry created from a file descriptor referring >>> a cgroup directory (e.g. by opening /sys/fs/cgroup/X). In this case, the >>> map entry is of type BPF_MAP_HANDLE_TYPE_LANDLOCK_CGROUP_FD and the >>> inferred array map is of type BPF_MAP_ARRAY_TYPE_LANDLOCK_CGROUP. >> >> Can you elaborate on why this is useful? I.e. why not just supply >> different policies to different subtrees. > > The main use case I see is to load the security policies at the start of > a user session for all processes but not enforce them right away. The > user can then keep a shell for Landlock administration tasks and lock > the other processes with a dedicated cgroup on the fly. This allows the > user to make unremovable Landlock security policies but only activate > them when needed for specific processes. This seems like a bit of a dubious use case to me. The landlock mechanism should be flexible enough to do this kind of thing even without cgroups, and "spawn a process, wait a while, and then confine it by fiddling with cgroups" seems a lot dicier than just loading the right policy in the first place, especially since eBPF policies can be stateful. > >> >> Also, how does this interact with the current cgroup v1 vs v2 mess? >> As far as I can tell, no one can even really agree on what "what >> cgroup am I in" means right now. > > I tested with cgroup-v2 but indeed, it seems a bit different with > cgroup-v1 :) > Does anyone know how to handle both cases? > >> >>> >>> An unprivileged process can create and manipulate cgroups thanks to >>> cgroup delegation. >> >> What is cgroup delegation? > > This is simply the action of changing the owner of cgroup sysfs files to > allow an unprivileged user to handle them (cf. Documentation/cgroup-v2.txt) As far as I can tell, Tejun and systemd both actively discourage doing this. Maybe I misunderstand. But in any event, the admin giving you a cgroup hierarchy you can use for this means that the admin has to cooperate with your policy, and it further requires (with cgroup v2 or similar, which is most likely the future) that your lockdown policy be compatible with your resource control policy. I would suggest dropping this lockdown feature until a use case emerges that really can't be addressed adequately without it. --Andy