[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