mirror of
https://github.com/checkpoint-restore/criu.git
synced 2026-07-22 17:49:13 +00:00
zdtm: don't destruct an external mount-namespace
Before doing something with mounts they should be marked as private, otherwise our actions can affects someone else. Signed-off-by: Andrey Vagin <avagin@openvz.org> Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
This commit is contained in:
parent
7aaef1ee4f
commit
55fe093908
1 changed files with 21 additions and 10 deletions
|
|
@ -1,3 +1,4 @@
|
|||
#include <stdbool.h>
|
||||
#include <string.h>
|
||||
#include <fcntl.h>
|
||||
#include <unistd.h>
|
||||
|
|
@ -18,11 +19,8 @@ static int test_fn(int argc, char **argv)
|
|||
FILE *f;
|
||||
int fd, tmpfs_fd;
|
||||
unsigned fs_cnt, fs_cnt_last = 0;
|
||||
bool private = false;
|
||||
|
||||
if (mount("none", "/", "none", MS_REC|MS_PRIVATE, NULL)) {
|
||||
err("Can't remount root with MS_PRIVATE");
|
||||
return -1;
|
||||
}
|
||||
again:
|
||||
fs_cnt = 0;
|
||||
f = fopen("/proc/self/mountinfo", "r");
|
||||
|
|
@ -41,18 +39,31 @@ again:
|
|||
end = strchr(mp, ' ');
|
||||
*end = '\0';
|
||||
|
||||
if (!strcmp(mp, "/"))
|
||||
continue;
|
||||
if (!strcmp(mp, "/proc"))
|
||||
continue;
|
||||
if (private) {
|
||||
if (!strcmp(mp, "/"))
|
||||
continue;
|
||||
if (!strcmp(mp, "/proc"))
|
||||
continue;
|
||||
|
||||
if (umount(mp))
|
||||
test_msg("umount(`%s') failed: %m\n", mp);
|
||||
if (umount(mp))
|
||||
test_msg("umount(`%s') failed: %m\n", mp);
|
||||
} else {
|
||||
/* mount --make-rprivate / */
|
||||
if (mount("none", mp, "none", MS_REC|MS_PRIVATE, NULL)) {
|
||||
err("Can't remount %s with MS_PRIVATE", mp);
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
fs_cnt++;
|
||||
}
|
||||
|
||||
fclose(f);
|
||||
|
||||
if (!private) {
|
||||
private = true;
|
||||
goto again;
|
||||
}
|
||||
|
||||
if (fs_cnt == 0)
|
||||
goto done;
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue