Hi Eric,

Today's linux-next merge of the userns tree got a conflict in
kernel/auditsc.c between commit e23eb920b0f3 ("audit: export
audit_log_task_info") from the security tree and commits e1760bd5ffae
("userns: Convert the audit loginuid  to be a kuid") and cca080d9b622
("userns: Convert audit to work with user namespaces enabled") from the
userns tree.

I fixed it up (I think - see below) and can carry the fix as necessary (no
action is required).

-- 
Cheers,
Stephen Rothwell                    s...@canb.auug.org.au

diff --cc kernel/auditsc.c
index 37f52f2,ff4798f..0000000
--- a/kernel/auditsc.c
+++ b/kernel/auditsc.c
@@@ -1160,32 -1151,8 +1152,38 @@@ void audit_log_task_info(struct audit_b
        char name[sizeof(tsk->comm)];
        struct mm_struct *mm = tsk->mm;
        struct vm_area_struct *vma;
 +      char *tty;
 +
 +      if (!ab)
 +              return;
  
        /* tsk == current */
 +      cred = current_cred();
 +
 +      spin_lock_irq(&tsk->sighand->siglock);
 +      if (tsk->signal && tsk->signal->tty && tsk->signal->tty->name)
 +              tty = tsk->signal->tty->name;
 +      else
 +              tty = "(none)";
 +      spin_unlock_irq(&tsk->sighand->siglock);
 +
 +
 +      audit_log_format(ab,
 +                       " ppid=%ld pid=%d auid=%u uid=%u gid=%u"
 +                       " euid=%u suid=%u fsuid=%u"
 +                       " egid=%u sgid=%u fsgid=%u ses=%u tty=%s",
 +                       sys_getppid(),
 +                       tsk->pid,
-                        tsk->loginuid, cred->uid, cred->gid,
-                        cred->euid, cred->suid, cred->fsuid,
-                        cred->egid, cred->sgid, cred->fsgid,
++                       from_kuid(&init_user_ns, tsk->loginuid),
++                       from_kuid(&init_user_ns, context->uid),
++                       from_kgid(&init_user_ns, context->gid),
++                       from_kuid(&init_user_ns, context->euid),
++                       from_kuid(&init_user_ns, context->suid),
++                       from_kuid(&init_user_ns, context->fsuid),
++                       from_kgid(&init_user_ns, context->egid),
++                       from_kgid(&init_user_ns, context->sgid),
++                       from_kgid(&init_user_ns, context->fsgid),
 +                       tsk->sessionid, tty);
  
        get_task_comm(name, tsk);
        audit_log_format(ab, " comm=");
@@@ -1208,10 -1175,8 +1206,10 @@@
        audit_log_task_context(ab);
  }
  
 +EXPORT_SYMBOL(audit_log_task_info);
 +
  static int audit_log_pid_context(struct audit_context *context, pid_t pid,
-                                uid_t auid, uid_t uid, unsigned int sessionid,
+                                kuid_t auid, kuid_t uid, unsigned int 
sessionid,
                                 u32 sid, char *comm)
  {
        struct audit_buffer *ab;

Attachment: pgpiOVEGqbVOf.pgp
Description: PGP signature

Reply via email to