[CRIU] [PATCH 2/7] mount: Add braces for safety sake

Cyrill Gorcunov gorcunov at openvz.org
Thu Sep 3 03:05:09 PDT 2015


Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
---
 mount.c | 39 ++++++++++++++++++++++++++-------------
 1 file changed, 26 insertions(+), 13 deletions(-)

diff --git a/mount.c b/mount.c
index a4e272341ed2..b7185c1b5175 100644
--- a/mount.c
+++ b/mount.c
@@ -52,9 +52,10 @@ static struct ext_mount *ext_mount_lookup(char *key)
 {
 	struct ext_mount *em;
 
-	list_for_each_entry(em, &opts.ext_mounts, list)
+	list_for_each_entry(em, &opts.ext_mounts, list) {
 		if (!strcmp(em->key, key))
 			return em;
+	}
 
 	return NULL;
 }
@@ -184,9 +185,10 @@ struct mount_info *lookup_overlayfs(char *rpath, unsigned int st_dev,
 	struct mount_info *m;
 
 	/* If the mnt_id and device number match for some entry, no fixup is needed */
-	for (m = mntinfo; m != NULL; m = m->next)
+	for (m = mntinfo; m != NULL; m = m->next) {
 		if (st_dev == m->s_dev && mnt_id == m->mnt_id)
 			return NULL;
+	}
 
 	return __lookup_overlayfs(mntinfo, rpath, st_dev, st_ino, mnt_id);
 }
@@ -195,9 +197,10 @@ static struct mount_info *__lookup_mnt_id(struct mount_info *list, int id)
 {
 	struct mount_info *m;
 
-	for (m = list; m != NULL; m = m->next)
+	for (m = list; m != NULL; m = m->next) {
 		if (m->mnt_id == id)
 			return m;
+	}
 
 	return NULL;
 }
@@ -211,9 +214,10 @@ struct mount_info *lookup_mnt_sdev(unsigned int s_dev)
 {
 	struct mount_info *m;
 
-	for (m = mntinfo; m != NULL; m = m->next)
+	for (m = mntinfo; m != NULL; m = m->next) {
 		if (m->s_dev == s_dev)
 			return m;
+	}
 
 	return NULL;
 }
@@ -378,9 +382,10 @@ static int mnt_depth(struct mount_info *m)
 	int depth = 0;
 	char *c;
 
-	for (c = m->mountpoint; *c != '\0'; c++)
+	for (c = m->mountpoint; *c != '\0'; c++) {
 		if (*c == '/')
 			depth++;
+	}
 
 	return depth;
 }
@@ -457,9 +462,10 @@ static struct mount_info *get_widest_peer(struct mount_info *m)
 	 * Try to find a mount, which is wider or equal.
 	 * A is wider than B, if A->root is a subpath of B->root.
 	 */
-	list_for_each_entry(p, &m->mnt_share, mnt_share)
+	list_for_each_entry(p, &m->mnt_share, mnt_share) {
 		if (issubpath(m->root, p->root))
 			return p;
+	}
 
 	return NULL;
 }
@@ -623,11 +629,12 @@ static struct mount_info *find_fsroot_mount_for(struct mount_info *bm)
 {
 	struct mount_info *sm;
 
-	list_for_each_entry(sm, &bm->mnt_bind, mnt_bind)
+	list_for_each_entry(sm, &bm->mnt_bind, mnt_bind) {
 		if (fsroot_mounted(sm) ||
 				(sm->parent == NULL &&
 				 strstartswith(bm->root, sm->root)))
 			return sm;
+	}
 
 	return NULL;
 }
@@ -796,13 +803,14 @@ static struct ns_id *find_ext_ns_id(void)
 	int pid = getpid();
 	struct ns_id *ns;
 
-	for (ns = ns_ids; ns->next; ns = ns->next)
+	for (ns = ns_ids; ns->next; ns = ns->next) {
 		if (ns->pid == pid && ns->nd == &mnt_ns_desc) {
 			if (!ns->mnt.mntinfo_list &&
 			    !collect_mntinfo(ns, true))
 				break;
 			return ns;
 		}
+	}
 
 	pr_err("Failed to find criu pid's mount ns!");
 	return NULL;
@@ -1666,9 +1674,10 @@ static int dump_mnt_ns(struct ns_id *ns, struct mount_info *pms)
 	if (!img)
 		goto err;
 
-	for (pm = pms; pm && pm->nsid == ns; pm = pm->next)
+	for (pm = pms; pm && pm->nsid == ns; pm = pm->next) {
 		if (dump_one_mountpoint(pm, img))
 			goto err_i;
+	}
 
 	ret = 0;
 err_i:
@@ -2085,16 +2094,19 @@ static bool can_mount_now(struct mount_info *mi)
 
 		if (mi->parent->shared_id == mi->shared_id) {
 			int rlen = strlen(mi->root);
-			list_for_each_entry(n, &p->mnt_share, mnt_share)
+			list_for_each_entry(n, &p->mnt_share, mnt_share) {
 				if (strlen(n->root) < rlen && !n->mounted)
 					return false;
+			}
 		} else {
-			list_for_each_entry(n, &p->mnt_share, mnt_share)
+			list_for_each_entry(n, &p->mnt_share, mnt_share) {
 				if (!n->mounted)
 					return false;
-			list_for_each_entry(n, &p->mnt_slave_list, mnt_slave)
+			}
+			list_for_each_entry(n, &p->mnt_slave_list, mnt_slave) {
 				if (!n->mounted)
 					return false;
+			}
 		}
 	}
 
@@ -2462,13 +2474,14 @@ static struct mount_info *read_mnt_ns_img(void)
 		if (nsid->nd != &mnt_ns_desc)
 			continue;
 
-		if (nsid->id != root_item->ids->mnt_ns_id)
+		if (nsid->id != root_item->ids->mnt_ns_id) {
 			/*
 			 * If we have more than one (root) namespace,
 			 * then we'll need the roots yard.
 			 */
 			if (create_mnt_roots())
 				return NULL;
+		}
 
 		if (collect_mnt_from_image(&pms, nsid))
 			return NULL;
-- 
2.4.3



More information about the CRIU mailing list