Sukadev Bhattiprolu [suka...@linux.vnet.ibm.com] wrote:
| Amerigo Wang [xiyou.wangc...@gmail.com] wrote:
| | >@@ -294,7 +295,11 @@ struct pid *alloc_pid(struct pid_namespace *ns)
| | > 
| | >   tmp = ns;
| | >   for (i = ns->level; i >= 0; i--) {
| | >-          nr = alloc_pidmap(tmp, 0);
| | >+          tpid = 0;
| | 
| | How about initializing 'tpid' to 0 when define it?
| 
| 'tpid' may contain a non-zero value from the previous iteration and
| needs to be reset each time.

Actually, you are right. target_pids does not change, so tpid could
be set to 0 before the loop.

Sukadev
_______________________________________________
Containers mailing list
contain...@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
Devel@openvz.org
https://openvz.org/mailman/listinfo/devel

Reply via email to