Hey Eric (Paris), this is the second time I've been notified of a merge issue with this audit patch; Is there something I need to do (or should have done earlier) to keep this from continuing to be an issue?
Cheers, peter On Mon, Sep 24, 2012 at 5:17 AM, Stephen Rothwell <s...@canb.auug.org.au> wrote: > Hi, > > On Mon, 24 Sep 2012 21:41:16 +1000 Stephen Rothwell <s...@canb.auug.org.au> > wrote: >> >> 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), > > These should all be "cred" not "context", of course. I fixed this in my > tree. > > -- > Cheers, > Stephen Rothwell s...@canb.auug.org.au -- Peter Moody Google 1.650.253.7306 Security Engineer pgp:0xC3410038 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
