[libvirt] [PATCH v3 11/12] LXC: controller: change the owner of /dev/pts and ptmx to the root of container
Richard RW. Weinberger
richard at sigma-star.at
Thu May 23 05:56:56 UTC 2013
Hi!
----- Ursprüngliche Mail -----
> This two files are created for container,
> the owner should be the root user of container.
>
> Signed-off-by: Gao feng <gaofeng at cn.fujitsu.com>
> ---
> src/lxc/lxc_controller.c | 30 ++++++++++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/src/lxc/lxc_controller.c b/src/lxc/lxc_controller.c
> index b2ace20..7d27135 100644
> --- a/src/lxc/lxc_controller.c
> +++ b/src/lxc/lxc_controller.c
> @@ -1506,8 +1506,15 @@
> virLXCControllerSetupDevPTS(virLXCControllerPtr ctrl)
> char *opts = NULL;
> char *devpts = NULL;
> char *path = NULL;
> + uid_t uid = (uid_t)-1;
> + gid_t gid = (gid_t)-1;
> int ret = -1;
>
> + if (ctrl->def->idmap.uidmap) {
> + uid = ctrl->def->idmap.uidmap[0].target;
> + gid = ctrl->def->idmap.gidmap[0].target;
> + }
> +
You're using this pattern a few times.
Maybe it's worth a helper function.
Thanks,
//richard
More information about the libvir-list
mailing list