Commit c3be52b2 by Serge Hallyn Committed by Stéphane Graber

chmod container dir to 0770

This prevents u2 from going into /home/u1/.local/share/lxc/u1/rootfs and running setuid-root applications to get write access to u1's container rootfs. Signed-off-by: 's avatarSerge Hallyn <serge.hallyn@ubuntu.com> Acked-by: 's avatarDwight Engen <dwight.engen@oracle.com>
parent c72e9abb
......@@ -738,6 +738,27 @@ static bool lxcapi_stop(struct lxc_container *c)
return ret == 0;
}
static int do_create_container_dir(const char *path, struct lxc_conf *conf)
{
int ret = -1;
char *p = alloca(strlen(path)+1);
ret = mkdir(path, 0770);
if (ret) {
if (errno == EEXIST)
ret = 0;
else {
SYSERROR("failed to create container path %s", path);
return -1;
}
}
strcpy(p, path);
if (!lxc_list_empty(&conf->id_map) && chown_mapped_root(p, conf) != 0) {
ERROR("Failed to chown container dir");
ret = -1;
}
return ret;
}
/*
* create the standard expected container dir
*/
......@@ -755,13 +776,7 @@ static bool create_container_dir(struct lxc_container *c)
free(s);
return false;
}
ret = mkdir(s, 0755);
if (ret) {
if (errno == EEXIST)
ret = 0;
else
SYSERROR("failed to create container path for %s", c->name);
}
ret = do_create_container_dir(s, c->lxc_conf);
free(s);
return ret == 0;
}
......@@ -2618,17 +2633,15 @@ sudo lxc-clone -o o1 -n n1 -s -L|-fssize fssize -v|--vgname vgname \
only rootfs gets converted (copied/snapshotted) on clone.
*/
static int create_file_dirname(char *path)
static int create_file_dirname(char *path, struct lxc_conf *conf)
{
char *p = strrchr(path, '/');
int ret;
int ret = -1;
if (!p)
return -1;
*p = '\0';
ret = mkdir(path, 0755);
if (ret && errno != EEXIST)
SYSERROR("creating container path %s", path);
ret = do_create_container_dir(path, conf);
*p = '/';
return ret;
}
......@@ -2672,7 +2685,7 @@ static struct lxc_container *lxcapi_clone(struct lxc_container *c, const char *n
goto out;
}
ret = create_file_dirname(newpath);
ret = create_file_dirname(newpath, c->lxc_conf);
if (ret < 0 && errno != EEXIST) {
ERROR("Error creating container dir for %s", newpath);
goto out;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment