chiark / gitweb /
audit: since nspawn now sets CAP_AUDIT_CONTROL for containers we cannot user this...
authorLennart Poettering <lennart@poettering.net>
Mon, 15 Apr 2013 19:55:12 +0000 (21:55 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 15 Apr 2013 19:59:04 +0000 (21:59 +0200)
As audit is still broken in containers we need a reliable way how we can
determine whether the audit data we read from 7proc is actually useful.
Previously we used CAP_AUDIT_CONTROL for this, since nspawn removed that
from the nspawn container. This has changed a while back however, which
means we used audit data of host system in the container.

This adds an explicit container check to the audit calls, so that all
audit data is turned off in containers.

This should fix session creation with pam_systemd/logind in nspawn containers.

src/shared/audit.c

index 138ca1d..97560cc 100644 (file)
@@ -34,6 +34,7 @@
 #include "util.h"
 #include "log.h"
 #include "fileio.h"
+#include "virt.h"
 
 int audit_session_from_pid(pid_t pid, uint32_t *id) {
         char *s;
@@ -45,6 +46,10 @@ int audit_session_from_pid(pid_t pid, uint32_t *id) {
         if (have_effective_cap(CAP_AUDIT_CONTROL) <= 0)
                 return -ENOENT;
 
+        /* Audit doesn't support containers right now */
+        if (detect_container(NULL) > 0)
+                return -ENOTSUP;
+
         if (pid == 0)
                 r = read_one_line_file("/proc/self/sessionid", &s);
         else {
@@ -90,6 +95,10 @@ int audit_loginuid_from_pid(pid_t pid, uid_t *uid) {
         if (have_effective_cap(CAP_AUDIT_CONTROL) <= 0)
                 return -ENOENT;
 
+        /* Audit doesn't support containers right now */
+        if (detect_container(NULL) > 0)
+                return -ENOTSUP;
+
         if (pid == 0)
                 r = read_one_line_file("/proc/self/loginuid", &s);
         else {