Quoting Eric W. Biederman (ebiederm@xxxxxxxxxxxx):
>
> In the -mm tree the rules for access an nsproxy have changed,
> and in get_net_ns_by_pid we access the nsproxy, so update
> it to follow the new rules.
>
> Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
Yup, looks right.
I assume Pavel's Acked-by would actually matter, but still
Acked-by: Serge Hallyn <serue@xxxxxxxxxx>
thanks,
-serge
> ---
>
> diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
> index 739fbad..1caba10 100644
> --- a/net/core/rtnetlink.c
> +++ b/net/core/rtnetlink.c
> @@ -746,10 +746,10 @@ static struct net *get_net_ns_by_pid(pid_t pid)
> rcu_read_lock();
> tsk = find_task_by_pid(pid);
> if (tsk) {
> - task_lock(tsk);
> - if (tsk->nsproxy)
> - net = get_net(tsk->nsproxy->net_ns);
> - task_unlock(tsk);
> + struct nsproxy *nsproxy;
> + nsproxy = task_nsproxy(tsk);
> + if (nsproxy)
> + net = get_net(nsproxy->net_ns);
> }
> rcu_read_unlock();
> return net;
> _______________________________________________
> Containers mailing list
> Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/containers
