[Devel] Re: [PATCH] pid: Extend/Fix pid_vnr

Serge E. Hallyn serue at us.ibm.com
Thu Dec 6 17:57:27 PST 2007


Quoting sukadev at us.ibm.com (sukadev at us.ibm.com):
> Oleg Nesterov [oleg at tv-sign.ru] wrote:
> | On 12/05, Eric W. Biederman wrote:
> | > 
> | > +pid_t pid_vnr(struct pid *pid)
> | > +{
> | > +	return pid_nr_ns(pid, current->nsproxy->pid_ns);
> | > +}
> 
> Hmm. current->nsproxy be NULL during process exit ?
> So this safe as long as pid_vnr() is not called after a
> process exits its namespaces. Probably no such callers atm. 

Yes I did a little audit for those this morning bc I frankly didn't
believe there weren't any.  But I couldn't find any :)

-serge

> | 
> | Excellent!!!
> | 
> | This allows us to do many cleanups. I am sending the trivial patch just
> | as example.
> | 
> | Oleg.
> _______________________________________________
> Containers mailing list
> Containers at lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list