lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 11 Nov 2009 14:39:43 -0800 From: Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com> To: Andrew Morton <akpm@...l.org> Cc: mtk.manpages@...glemail.com, arnd@...db.de, Containers <containers@...ts.linux-foundation.org>, Nathan Lynch <nathanl@...tin.ibm.com>, matthltc@...a.localdomain, "Eric W. Biederman" <ebiederm@...ssion.com>, hpa@...or.com, linux-api@...r.kernel.org, Alexey Dobriyan <adobriyan@...il.com>, roland@...hat.com, Pavel Emelyanov <xemul@...nvz.org>, linux-kernel@...r.kernel.org Subject: Re: [v12][PATCH 4/9] Add target_pids parameter to alloc_pid() Cc: LKML Sukadev Bhattiprolu [sukadev@...ux.vnet.ibm.com] wrote: | | Subject: [v12][PATCH 4/9] Add target_pids parameter to alloc_pid() | | This parameter is currently NULL, but will be used in a follow-on patch. | | Signed-off-by: Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com> | Acked-by: Serge Hallyn <serue@...ibm.com> | Reviewed-by: Oren Laadan <orenl@...columbia.edu> | --- | include/linux/pid.h | 2 +- | kernel/fork.c | 3 ++- | kernel/pid.c | 9 +++++++-- | 3 files changed, 10 insertions(+), 4 deletions(-) | | diff --git a/include/linux/pid.h b/include/linux/pid.h | index 49f1c2f..914185d 100644 | --- a/include/linux/pid.h | +++ b/include/linux/pid.h | @@ -119,7 +119,7 @@ extern struct pid *find_get_pid(int nr); | extern struct pid *find_ge_pid(int nr, struct pid_namespace *); | int next_pidmap(struct pid_namespace *pid_ns, int last); | | -extern struct pid *alloc_pid(struct pid_namespace *ns); | +extern struct pid *alloc_pid(struct pid_namespace *ns, pid_t *target_pids); | extern void free_pid(struct pid *pid); | | /* | diff --git a/kernel/fork.c b/kernel/fork.c | index 93626b2..3f1dddf 100644 | --- a/kernel/fork.c | +++ b/kernel/fork.c | @@ -980,6 +980,7 @@ static struct task_struct *copy_process(unsigned long clone_flags, | int retval; | struct task_struct *p; | int cgroup_callbacks_done = 0; | + pid_t *target_pids = NULL; | | if ((clone_flags & (CLONE_NEWNS|CLONE_FS)) == (CLONE_NEWNS|CLONE_FS)) | return ERR_PTR(-EINVAL); | @@ -1156,7 +1157,7 @@ static struct task_struct *copy_process(unsigned long clone_flags, | goto bad_fork_cleanup_io; | | if (pid != &init_struct_pid) { | - pid = alloc_pid(p->nsproxy->pid_ns); | + pid = alloc_pid(p->nsproxy->pid_ns, target_pids); | if (IS_ERR(pid)) { | retval = PTR_ERR(pid); | goto bad_fork_cleanup_io; | diff --git a/kernel/pid.c b/kernel/pid.c | index afef766..3ee52cd 100644 | --- a/kernel/pid.c | +++ b/kernel/pid.c | @@ -281,13 +281,14 @@ void free_pid(struct pid *pid) | call_rcu(&pid->rcu, delayed_put_pid); | } | | -struct pid *alloc_pid(struct pid_namespace *ns) | +struct pid *alloc_pid(struct pid_namespace *ns, pid_t *target_pids) | { | struct pid *pid; | enum pid_type type; | int i, nr; | struct pid_namespace *tmp; | struct upid *upid; | + pid_t tpid; | | pid = kmem_cache_alloc(ns->pid_cachep, GFP_KERNEL); | if (!pid) { | @@ -297,7 +298,11 @@ struct pid *alloc_pid(struct pid_namespace *ns) | | tmp = ns; | for (i = ns->level; i >= 0; i--) { | - nr = alloc_pidmap(tmp); | + tpid = 0; | + if (target_pids) | + tpid = target_pids[i]; | + | + nr = set_pidmap(tmp, tpid); | if (nr < 0) | goto out_free; | | -- | 1.6.0.4 | | _______________________________________________ | Containers mailing list | Containers@...ts.linux-foundation.org | https://lists.linux-foundation.org/mailman/listinfo/containers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists