From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Guy Briggs Subject: [PATCH 0/5][RFC][v2] steps to make audit pid namespace-safe Date: Mon, 23 Dec 2013 17:27:34 -0500 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-audit-bounces@redhat.com Errors-To: linux-audit-bounces@redhat.com To: linux-audit@redhat.com, linux-kernel@vger.kernel.org Cc: John Johansen , Richard Guy Briggs , Oleg Nesterov , Peter Zijlstra , Ingo Molnar , "Eric W. Biederman" List-Id: linux-audit@redhat.com This patchset is a revival of some of Eric Biederman's work to make audit pid-namespace-safe. In a couple of places, audit was printing PIDs in the task's pid namespace rather than relative to the audit daemon's pid namespace, which currently is init_pid_ns. It also allows processes to log audit user messages in their own pid namespaces, which was not previously permitted. Please see: https://bugzilla.redhat.com/show_bug.cgi?id=947530 https://bugs.launchpad.net/ubuntu/+source/vsftpd/+bug/1160372 https://bugzilla.novell.com/show_bug.cgi?id=786024 Part of the cleanup here involves deprecating task->pid and task->tgid, which should be accessed using their respective helper functions. See: https://lkml.org/lkml/2013/8/20/638 Richard Guy Briggs (5): pid: get pid_t ppid of task in init_pid_ns audit: convert PPIDs to the inital PID namespace. audit: store audit_pid as a struct pid pointer audit: anchor all pid references in the initial pid namespace audit: allow user processes to log from another PID namespace drivers/tty/tty_audit.c | 3 +- include/linux/sched.h | 24 +++++++++++++++ kernel/audit.c | 54 ++++++++++++++++++++++++---------- kernel/audit.h | 4 +- kernel/auditfilter.c | 17 ++++++++++- kernel/auditsc.c | 24 ++++++++------- security/apparmor/audit.c | 2 +- security/integrity/integrity_audit.c | 2 +- security/lsm_audit.c | 11 ++++-- 9 files changed, 104 insertions(+), 37 deletions(-)