From: Daniel Lezcano Date: Thu, 13 Jan 2011 15:25:14 +0000 (+0100) Subject: substitute the absolute rootfs mount path X-Git-Tag: lxc-0.7.4-rc1~17 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=013bd428482534e6de00b385b0a501c8c71c661b;p=thirdparty%2Flxc.git substitute the absolute rootfs mount path Change the mount point in the rootfs because we mount the rootfs in ROOTFSDIR for the pivot. We have to substitute the real mount path to the new path located in ROOTFSDIR. Signed-off-by: Daniel Lezcano --- diff --git a/src/lxc/conf.c b/src/lxc/conf.c index 345facda6..df6f485a3 100644 --- a/src/lxc/conf.c +++ b/src/lxc/conf.c @@ -959,28 +959,32 @@ static inline int mount_entry_on_systemfs(struct mntent *mntent) } static int mount_entry_on_absolute_rootfs(struct mntent *mntent, - const char *rootfs) + const struct lxc_rootfs *rootfs) { + char *aux; char path[MAXPATHLEN]; unsigned long mntflags; char *mntdata; - int ret; + int ret = 0; if (parse_mntopts(mntent->mnt_opts, &mntflags, &mntdata) < 0) { ERROR("failed to parse mount option '%s'", mntent->mnt_opts); return -1; } - if (strncmp(mntent->mnt_dir, rootfs, strlen(rootfs))) - WARN("mount target directory '%s' is outside " - "container root", mntent->mnt_dir); - else - WARN("mount target directory '%s' is not " - "relative to container root", mntent->mnt_dir); + aux = strstr(mntent->mnt_dir, rootfs->path); + if (!aux) { + WARN("ignoring mount point '%s'", mntent->mnt_dir); + goto out; + } + + snprintf(path, MAXPATHLEN, "%s%s", rootfs->mount, + aux + strlen(rootfs->path)); - ret = mount_entry(mntent->mnt_fsname, mntent->mnt_dir, mntent->mnt_type, + ret = mount_entry(mntent->mnt_fsname, path, mntent->mnt_type, mntflags, mntdata); +out: free(mntdata); return ret; } @@ -1030,7 +1034,7 @@ static int mount_file_entries(const struct lxc_rootfs *rootfs, FILE *file) continue; } - if (mount_entry_on_absolute_rootfs(mntent, rootfs->path)) + if (mount_entry_on_absolute_rootfs(mntent, rootfs)) goto out; }