Fix a warning in api.c, where we ignore the fscanf value.

I've tested the patch by running the tests available in the tests
directory.

Signed-off-by: Balbir Singh <balbir@linux.vnet.ibm.com>



git-svn-id: https://libcg.svn.sourceforge.net/svnroot/libcg/trunk@185 4f4bb910-9a46-0410-90c8-c897d4f1cd53

1 files changed, 6 insertions, 1 deletions

diff --git a/api.c b/api.c
index 99037e8..fef34c1 100644
--- a/api.c
+++ b/api.c
@@ -902,6 +902,7 @@ static char *cg_rd_ctrl_file(char *subsys, char *cgroup, char *file)
 	char *value;
 	char path[FILENAME_MAX];
 	FILE *ctrl_file;
+	int ret;
 
 	if (!cg_build_path_locked(cgroup, path, subsys))
 		return NULL;
@@ -919,7 +920,11 @@ static char *cg_rd_ctrl_file(char *subsys, char *cgroup, char *file)
 	 * using %as crashes when we try to read from files like
 	 * memory.stat
 	 */
-	fscanf(ctrl_file, "%s", value);
+	ret = fscanf(ctrl_file, "%s", value);
+	if (ret == 0 || ret == EOF) {
+		free(value);
+		value = NULL;
+	}
 
 	fclose(ctrl_file);