Re: [RFC 4/5] user namespaces: allow killing tasks in your own orchild userns

From: Serge E. Hallyn
Date: Fri Dec 17 2010 - 15:22:07 EST


Quoting Eric W. Biederman (ebiederm@xxxxxxxxxxxx):
> >> > cred = current_cred();
> >> > tcred = __task_cred(t);
> >> Nit pick you don't need to compute cred and tcred here now.
> >
> > Just to make sure I understand right: you mean wait until after the
> > same_thread_group() check to save calculation in that case, right?
>
> I mean cred and tcred are only use in kill_ok_by_cred.
> So we can eliminate those two variables from check_kill_permission.

D'oh. Should've looked at the original tree, not the context. Got it,
thanks.

-serge
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/