From: Andrii Simiklit <andrii.simik...@globallogic.com>
1. tools/i965_disasm.c:58:4: warning:
ignoring return value of ‘fread’,
declared with attribute warn_unused_result
fread(assembly, *end, 1, fp);
v2: - Fixed incorrect return value check.
( Eric Engestrom <eric.engest...@intel.com> )
Signed-off-by: Andrii Simiklit <andrii.simik...@globallogic.com>
---
src/intel/tools/i965_disasm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/intel/tools/i965_disasm.c b/src/intel/tools/i965_disasm.c
index 73a6760fc1..329f6327ed 100644
--- a/src/intel/tools/i965_disasm.c
+++ b/src/intel/tools/i965_disasm.c
@@ -55,7 +55,8 @@ i965_disasm_read_binary(FILE *fp, size_t *end)
if (assembly == NULL)
return NULL;
- fread(assembly, *end, 1, fp);
+ MAYBE_UNUSED size_t size = fread(assembly, *end, 1, fp);
+ assert((size || (*end == 0)) && "error: unable to read all elements!");
fclose(fp);
return assembly;
--
2.17.1
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
