From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Lutomirski Subject: Re: [PATCH V6 05/10] audit: log creation and deletion of namespace instances Date: Fri, 15 May 2015 06:17:31 -0700 Message-ID: References: <20150515023221.GC965@madcap2.tricolour.ca> <4153072.0EnVzNaVVH@x2> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4153072.0EnVzNaVVH@x2> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Steve Grubb Cc: "Eric W. Biederman" , Linux API , Linux Containers , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Paul Moore , linux-audit-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, Al Viro , Network Development , Linux FS Devel , Eric Paris , Mimi Zohar List-Id: linux-api@vger.kernel.org On May 15, 2015 9:38 PM, "Steve Grubb" wrote: > > On Thursday, May 14, 2015 11:23:09 PM Andy Lutomirski wrote: > > On Thu, May 14, 2015 at 7:32 PM, Richard Guy Briggs wrote: > > > On 15/05/14, Paul Moore wrote: > > >> * Look at our existing audit records to determine which records should > > >> have > > >> namespace and container ID tokens added. We may only want to add the > > >> additional fields in the case where the namespace/container ID tokens are > > >> not the init namespace. > > > > > > If we have a record that ties a set of namespace IDs with a container > > > ID, then I expect we only need to list the containerID along with auid > > > and sessionID. > > > > The problem here is that the kernel has no concept of a "container", and I > > don't think it makes any sense to add one just for audit. "Container" is a > > marketing term used by some userspace tools. > > No, its a real thing just like a login. Does the kernel have any concept of a > login? Yet it happens. And it causes us to generate events describing who, > where from, role, success, and time of day. :-) > I really hope those records come from userspace, not the kernel. I also wonder what happens when a user logs in and types "sudo agetty /dev/ttyS0 115200". If a user does that and then someone logs in on /dev/ttyS0, which login are they? > > > I can imagine that both audit could benefit from a concept of a > > namespace *path* that understands nesting (e.g. root/2/5/1 or > > something along those lines). Mapping these to "containers" belongs > > in userspace, I think. > > I don't doubt that just as user space sequences the actions that are a login. > I just need the kernel to do some book keeping and associate the necessary > attributes in the event record to be able to reconstruct what is actually > happening. A precondition for that is having those records have some correspondence to what is actually happening. Since the kernel has no concept of a container, and since the same kernel mechanisms could be used for things that are probably not whatever the Common Criteria rules think a container is, this could be quite difficult to define in a meaningful manner. Hence my suggestion to add only minimal support in the kernel and to do this in userspace. --Andy