cgroup: Introduce restore_special_propery()

To reduce restore_special_props() size - let's factor a new
function for restoring one special property.

No functional change expected.
Signed-off-by: Dmitry Safonov <dsafonov@virtuozzo.com>
Signed-off-by: Andrei Vagin <avagin@virtuozzo.com>

criu/cgroup.c

@@ -1371,72 +1371,57 @@ int prepare_cgroup_properties(void)
 	return 0;
 }
 
-static int restore_special_props(char *paux, size_t off, CgroupDirEntry *e)
+static int restore_special_property(char *paux, size_t off, CgroupPropEntry *pr)
 {
-	int j;
-
-	pr_info("Restore special props\n");
-
-	{
-
-		for (j = 0; j < e->n_properties; j++) {
-			CgroupPropEntry *prop = e->properties[j];
-
-			if (is_special_property(prop->name)) {
-				/* XXX: we can drop this hack and make
-				 * memory.swappiness and memory.oom_control
-				 * regular properties when we drop support for
+	/*
+	 * XXX: we can drop this hack and make memory.swappiness and
+	 * memory.oom_control regular properties when we drop support for
 	 * kernels < 3.16. See 3dae7fec5.
 	 */
-				if (!strcmp(prop->name, "memory.swappiness") &&
-						!strcmp(prop->value, "60")) {
-					continue;
-				} else if (!strcmp(prop->name, "memory.oom_control") &&
-						!strcmp(prop->value, "0")) {
-					continue;
-				}
+	if (!strcmp(pr->name, "memory.swappiness") && !strcmp(pr->value, "60"))
+		return 0;
+	if (!strcmp(pr->name, "memory.oom_control") && !strcmp(pr->value, "0"))
+		return 0;
 
-				if (!strcmp(prop->name, "devices.list")) {
-					/* The devices cgroup must be restored in a
-					 * special way: only the contents of
-					 * devices.list can be read, and it is a
-					 * whitelist of all the devices the cgroup is
-					 * allowed to create. To re-creat this
-					 * whitelist, we first deny everything via
-					 * devices.deny, and then write the list back
+	if (!strcmp(pr->name, "devices.list")) {
+		/*
+		 * The devices cgroup must be restored in a special way:
+		 * only the contents of devices.list can be read, and it
+		 * is a whitelist of all the devices the cgroup is allowed
+		 * to create. To re-create this whitelist, we firstly deny
+		 * everything via devices.deny, and then write the list back
 		 * into devices.allow.
 		 *
-					 * Further, we must have a write() call for
-					 * each line, because the kernel only parses
-					 * the first line of any write().
+		 * Further, we must have a write() call for each line,
+		 * because the kernel only parses the first line of
+		 * any write().
 		 */
-					CgroupPropEntry *pe = prop;
+		CgroupPropEntry *pe = pr;
 		char *old_val = pe->value, *old_name = pe->name;
 		int ret;
 		char *pos;
 
-					/* A bit of a fudge here. These are
-					 * write only by owner by default, but
-					 * the container engine could have
-					 * changed the perms. We should come up
-					 * with a better way to restore all of
-					 * this stuff.
+		/*
+		 * A bit of a fudge here. These are write only by owner
+		 * by default, but the container engine could have changed
+		 * the perms. We should come up with a better way to
+		 * restore all of this stuff.
 		 */
 		pe->perms->mode = 0200;
 
 		pe->name = "devices.deny";
 		pe->value = "a";
-					ret = restore_cgroup_prop(prop, paux, off);
+		ret = restore_cgroup_prop(pr, paux, off);
 		pe->name = old_name;
 		pe->value = old_val;
 
-					/* an emptry string here means nothing
-					 * is allowed, and the kernel disallows
-					 * writing an "" to devices.allow, so
-					 * let's just keep going.
+		/*
+		 * An emptry string here means nothing is allowed,
+		 * and the kernel disallows writing an "" to devices.allow,
+		 * so let's just keep going.
 		 */
 		if (!strcmp(pe->value, ""))
-						continue;
+			return 0;
 
 		if (ret < 0)
 			return -1;
@@ -1457,16 +1442,29 @@ static int restore_special_props(char *paux, size_t off, CgroupDirEntry *e)
 		}
 		pe->value = old_val;
 		pe->name = old_name;
-					continue;
-
+		return 0;
 	}
 
-				if (restore_cgroup_prop(prop, paux, off) < 0) {
+	return restore_cgroup_prop(pr, paux, off);
+}
+
+static int restore_special_props(char *paux, size_t off, CgroupDirEntry *e)
+{
+	unsigned int j;
+
+	pr_info("Restore special props\n");
+
+	for (j = 0; j < e->n_properties; j++) {
+		CgroupPropEntry *prop = e->properties[j];
+
+		if (!is_special_property(prop->name))
+			continue;
+
+		if (restore_special_property(paux, off, prop) < 0) {
+			pr_err("Restoring %s special property failed\n", prop->name);
 			return -1;
 		}
 	}
-		}
-	}
 
 	return 0;
 }