[CRIU] [PATCH] mount: fix segmentation fault in can_mount_now on no mnt_master
Pavel Tikhomirov
ptikhomirov at virtuozzo.com
Tue Jul 24 12:55:02 MSK 2018
In resolve_shared_mounts there are cases when we have m->master_id > 0
but m->mnt_master is not set, it happens for cases where we have no
access to master mount, for instance CT root m->parent==NULL or when
mount is external. In can_mount_now for such master mounts we don't
need to check mounted state also, so just use "if(mi->mnt_master)"
condition instead of "if(mi->master_id > 0)" to fix segfault.
https://jira.sw.ru/browse/PSBM-86978
Program terminated with signal 11, Segmentation fault.
0x000000000046328b in can_mount_now (mi=0x2155970) at criu/mount.c:2699
2699 list_for_each_entry(c, &mi->mnt_master->children, siblings)
(gdb) p mi->mnt_master
$2 = (struct mount_info *) 0x0
Fixes commit 3a02362c5be1 ("mount: fix can_mount_now to wait children of
master's share properly")
Signed-off-by: Pavel Tikhomirov <ptikhomirov at virtuozzo.com>
---
criu/mount.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/criu/mount.c b/criu/mount.c
index 9cc8f6e94..d2292cd8e 100644
--- a/criu/mount.c
+++ b/criu/mount.c
@@ -2355,7 +2355,7 @@ static bool can_mount_now(struct mount_info *mi)
* - Make sure all children of master's share are
* mounted as well to eliminate mounts duplications
*/
- if (mi->master_id > 0) {
+ if (mi->mnt_master) {
struct mount_info *c, *s;
if (mi->bind == NULL)
--
2.17.1
More information about the CRIU
mailing list