Blame SOURCES/libcgroup-0.40.rc1-fread.patch

308897
diff -up libcgroup-0.41/src/api.c.fread libcgroup-0.41/src/api.c
308897
--- libcgroup-0.41/src/api.c.fread	2014-01-13 21:01:32.067067615 +0100
308897
+++ libcgroup-0.41/src/api.c	2014-01-13 21:01:32.070067594 +0100
308897
@@ -2232,29 +2232,29 @@ static int cg_rd_ctrl_file(const char *s
308897
 					const char *file, char **value)
308897
 {
308897
 	char path[FILENAME_MAX];
308897
-	FILE *ctrl_file = NULL;
308897
-	int ret;
308897
+	int ctrl_file = -1;
308897
+	ssize_t ret;
308897
 
308897
 	if (!cg_build_path_locked(cgroup, path, subsys))
308897
 		return ECGFAIL;
308897
 
308897
 	strncat(path, file, sizeof(path) - strlen(path));
308897
-	ctrl_file = fopen(path, "re");
308897
-	if (!ctrl_file)
308897
+	ctrl_file = open(path, O_RDONLY | O_CLOEXEC);
308897
+	if (ctrl_file < 0)
308897
 		return ECGROUPVALUENOTEXIST;
308897
 
308897
 	*value = calloc(CG_VALUE_MAX, 1);
308897
 	if (!*value) {
308897
-		fclose(ctrl_file);
308897
+		close(ctrl_file);
308897
 		last_errno = errno;
308897
 		return ECGOTHER;
308897
 	}
308897
 
308897
 	/*
308897
-	 * using %as crashes when we try to read from files like
308897
+	 * using %as or fread crashes when we try to read from files like
308897
 	 * memory.stat
308897
 	 */
308897
-	ret = fread(*value, 1, CG_VALUE_MAX-1, ctrl_file);
308897
+	ret = read(ctrl_file, *value, CG_VALUE_MAX-1);
308897
 	if (ret < 0) {
308897
 		free(*value);
308897
 		*value = NULL;
308897
@@ -2264,7 +2264,7 @@ static int cg_rd_ctrl_file(const char *s
308897
 			(*value)[ret-1] = '\0';
308897
 	}
308897
 
308897
-	fclose(ctrl_file);
308897
+	close(ctrl_file);
308897
 
308897
 	return 0;
308897
 }