OpenVZ Forum


Home » Mailing lists » Devel » [PATCH] pid: Extend/Fix pid_vnr
Re: [PATCH] pid: Extend/Fix pid_vnr [message #24682 is a reply to message #24624] Fri, 07 December 2007 18:10 Go to previous messageGo to previous message
ebiederm is currently offline  ebiederm
Messages: 1354
Registered: February 2006
Senior Member
"Serge E. Hallyn" <serue@us.ibm.com> writes:

> Quoting sukadev@us.ibm.com (sukadev@us.ibm.com):
>> Oleg Nesterov [oleg@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 :)

Cool.  The only case pid_vnr would make sense in that context is if
we were talking to user space after we had exited our namespaces.
Which is at least as stretch.

Eric
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: [PATCH][VLAN] Merge tree equal tails in vlan_skb_recv
Next Topic: [PATCH][IPV4] Swap the ifa allocation with the"ipv4_devconf_setall" call
Goto Forum:
  


Current Time: Tue Aug 26 20:17:39 GMT 2025

Total time taken to generate the page: 0.15332 seconds