From 459b8719590bb2a02bccdf4d2690a32e2d01d229 Mon Sep 17 00:00:00 2001 From: David Ward Date: Tue, 23 Jun 2015 10:57:33 -0400 Subject: [PATCH] Only mount /proc if needed, even without a rootfs Use the same code with and without a rootfs to check if mounting /proc is necessary before doing so. If mounting it is unsuccessful and there is no rootfs, continue as before. Signed-off-by: David Ward Acked-by: Serge E. Hallyn --- src/lxc/conf.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/lxc/conf.c b/src/lxc/conf.c index d01a76bd9..1d1cfb2ed 100644 --- a/src/lxc/conf.c +++ b/src/lxc/conf.c @@ -3994,17 +3994,12 @@ int tmp_proc_mount(struct lxc_conf *lxc_conf) { int mounted; - if (lxc_conf->rootfs.path == NULL || strlen(lxc_conf->rootfs.path) == 0) { - if (mount("proc", "/proc", "proc", 0, NULL)) { - SYSERROR("Failed mounting /proc, proceeding"); - mounted = 0; - } else - mounted = 1; - } else - mounted = do_tmp_proc_mount(lxc_conf->rootfs.mount); + mounted = do_tmp_proc_mount(lxc_conf->rootfs.path ? lxc_conf->rootfs.mount : ""); if (mounted == -1) { SYSERROR("failed to mount /proc in the container."); - return -1; + /* continue only if there is no rootfs */ + if (lxc_conf->rootfs.path) + return -1; } else if (mounted == 1) { lxc_conf->tmp_umount_proc = 1; } -- 2.47.2