[Intel-gfx] [PATCH 7/8] fix warn in prime_nv_*: ignoring return value of ‘fgets’

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


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

diff --git a/tests/prime_nv_api.c b/tests/prime_nv_api.c
index 962e903..e1b7117 100644
--- a/tests/prime_nv_api.c
+++ b/tests/prime_nv_api.c
@@ -38,6 +38,8 @@ static int find_and_open_devices(void)
 	char vendor_id[8];
 	int venid;
 	for (i = 0; i < 9; i++) {
+		char *ret;
+
 		sprintf(path, "/sys/class/drm/card%d/device/vendor", i);
 		if (stat(path, &buf))
 			break;
@@ -46,7 +48,8 @@ static int find_and_open_devices(void)
 		if (!fl)
 			break;
 
-		fgets(vendor_id, 8, fl);
+		ret = fgets(vendor_id, 8, fl);
+		assert(ret);
 		fclose(fl);
 
 		venid = strtoul(vendor_id, NULL, 16);
diff --git a/tests/prime_nv_test.c b/tests/prime_nv_test.c
index 2269f84..f9a766a 100644
--- a/tests/prime_nv_test.c
+++ b/tests/prime_nv_test.c
@@ -46,6 +46,8 @@ static int find_and_open_devices(void)
 	char vendor_id[8];
 	int venid;
 	for (i = 0; i < 9; i++) {
+		char *ret;
+
 		sprintf(path, "/sys/class/drm/card%d/device/vendor", i);
 		if (stat(path, &buf))
 			break;
@@ -54,7 +56,8 @@ static int find_and_open_devices(void)
 		if (!fl)
 			break;
 
-		fgets(vendor_id, 8, fl);
+		ret = fgets(vendor_id, 8, fl);
+		assert(ret);
 		fclose(fl);
 
 		venid = strtoul(vendor_id, NULL, 16);
-- 
1.7.9.5




More information about the Intel-gfx mailing list