[Intel-gfx] [PATCH 5/8] fix warn in sysfs_{rc6, rps}*: ignoring return value of 'fscanf'

Imre Deak imre.deak at intel.com
Wed Oct 10 15:04:42 CEST 2012


Signed-off-by: Imre Deak <imre.deak at intel.com>
---
 tests/sysfs_rc6_residency.c |    5 ++++-
 tests/sysfs_rps.c           |    6 +++++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/tests/sysfs_rc6_residency.c b/tests/sysfs_rc6_residency.c
index 2f33697..cd62e77 100644
--- a/tests/sysfs_rc6_residency.c
+++ b/tests/sysfs_rc6_residency.c
@@ -38,6 +38,7 @@
 static unsigned int readit(const char *path)
 {
 	unsigned int ret;
+	int scanned;
 
 	FILE *file;
 	file = fopen(path, "r");
@@ -45,7 +46,9 @@ static unsigned int readit(const char *path)
 		fprintf(stderr, "Couldn't open %s (%d)\n", path, errno);
 		abort();
 	}
-	fscanf(file, "%u", &ret);
+	scanned = fscanf(file, "%u", &ret);
+	assert(scanned == 1);
+
 	fclose(file);
 
 	return ret;
diff --git a/tests/sysfs_rps.c b/tests/sysfs_rps.c
index b830797..833c2e9 100644
--- a/tests/sysfs_rps.c
+++ b/tests/sysfs_rps.c
@@ -64,9 +64,13 @@ struct junk {
 static int readval(FILE *filp)
 {
 	int val;
+	int scanned;
+
 	fflush(filp);
 	rewind(filp);
-	fscanf(filp, "%d", &val);
+	scanned = fscanf(filp, "%d", &val);
+	assert(scanned == 1);
+
 	return val;
 }
 
-- 
1.7.9.5




More information about the Intel-gfx mailing list