[CRIU] [PATCH 2/2] mount: don't skip checks in validate_mounts()

Andrey Vagin avagin at openvz.org
Wed Sep 10 05:46:41 PDT 2014


"continue" is called by mistake, so we skip a few checks for shared
mounts without siblings.

Signed-off-by: Andrey Vagin <avagin at openvz.org>
---
 mount.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/mount.c b/mount.c
index 585375c..bf27052 100644
--- a/mount.c
+++ b/mount.c
@@ -377,10 +377,9 @@ static int validate_mounts(struct mount_info *info, bool for_dump)
 			/* root mount can be any */
 			continue;
 
-		if (m->parent->shared_id) {
+		if (m->parent->shared_id && !list_empty(&m->parent->mnt_share)) {
 			struct mount_info *ct;
-			if (list_empty(&m->parent->mnt_share))
-				continue;
+
 			t = list_first_entry(&m->parent->mnt_share, struct mount_info, mnt_share);
 
 			list_for_each_entry(ct, &t->children, siblings) {
-- 
1.9.3



More information about the CRIU mailing list