Serge E. Hallyn <[email protected]> wrote:

> +int same_or_ancestor_user_ns(struct task_struct *task,
> +     struct task_struct *victim)
> +{
> +     struct user_namespace *u1 = task_cred_xxx(task, user)->user_ns;
> +     struct user_namespace *u2 = task_cred_xxx(victim, user)->user_ns;

Hmmm.  task_cred_xxx() uses task->real_cred, which is correct for victim (the
object), but normally you'd use task->cred for task (the subject).  However,
in this case, I think it's probably okay.

David
_______________________________________________
Containers mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
[email protected]
https://openvz.org/mailman/listinfo/devel

Reply via email to