[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