confile: fix setting prlimits

src/lxc/conf.h

@@ -104,7 +104,7 @@ struct lxc_limit {
 static void free_lxc_limit(struct lxc_limit *ptr)
 {
 	if (ptr) {
-		free(ptr->resource);
+		free_disarm(ptr->resource);
 		free_disarm(ptr);
 	}
 }

src/lxc/confile.c

@@ -1727,8 +1727,8 @@ static bool parse_limit_value(const char **value, rlim_t *res)
 static int set_config_prlimit(const char *key, const char *value,
 			    struct lxc_conf *lxc_conf, void *data)
 {
-	__do_free struct lxc_list *limlist = NULL;
-	call_cleaner(free_lxc_limit) struct lxc_limit *limelem = NULL;
+	__do_free struct lxc_list *list = NULL;
+	call_cleaner(free_lxc_limit) struct lxc_limit *elem = NULL;
 	struct lxc_list *iter;
 	struct rlimit limit;
 	rlim_t limit_value;
@@ -1779,29 +1779,31 @@ static int set_config_prlimit(const char *key, const char *value,
 
 	/* find existing list element */
 	lxc_list_for_each(iter, &lxc_conf->limits) {
-		limelem = iter->elem;
-		if (strequal(key, limelem->resource)) {
-			limelem->limit = limit;
-			return 0;
-		}
+		struct lxc_limit *cur = iter->elem;
+
+		if (!strequal(key, cur->resource))
+			continue;
+
+		cur->limit = limit;
+		return 0;
 	}
 
 	/* allocate list element */
-	limlist = lxc_list_new();
-	if (!limlist)
+	list = lxc_list_new();
+	if (!list)
 		return ret_errno(ENOMEM);
 
-	limelem = zalloc(sizeof(*limelem));
-	if (!limelem)
+	elem = zalloc(sizeof(*elem));
+	if (!elem)
 		return ret_errno(ENOMEM);
 
-	limelem->resource = strdup(key);
-	if (!limelem->resource)
+	elem->resource = strdup(key);
+	if (!elem->resource)
 		return ret_errno(ENOMEM);
 
-	limelem->limit = limit;
-	lxc_list_add_elem(limlist, move_ptr(limelem));;
-	lxc_list_add_tail(&lxc_conf->limits, move_ptr(limlist));
+	elem->limit = limit;
+	lxc_list_add_elem(list, move_ptr(elem));;
+	lxc_list_add_tail(&lxc_conf->limits, move_ptr(list));
 
 	return 0;
 }