summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean Sacren <sakiwit@gmail.com>2010-09-10 23:13:33 -0600
committerMichal Marek <mmarek@suse.cz>2010-09-29 16:18:27 +0200
commitef894870c6c38580e540c29cfb22a827d54a950a (patch)
treeb1065bc4e2e637aa2c80a07040636b0e9a98de7e
parent60eab877a9581b479c6c7d7913ce8070a0ace960 (diff)
downloadop-kernel-dev-ef894870c6c38580e540c29cfb22a827d54a950a.zip
op-kernel-dev-ef894870c6c38580e540c29cfb22a827d54a950a.tar.gz
scripts/kallsyms: Enable error messages while hush up unnecessary warnings
As no error was handled, we wouldn't be able to know when an error does occur. The fix preserves error messages while it doesn't let unnecessary compiling warnings show up. Signed-off-by: Jean Sacren <sakiwit@gmail.com> Signed-off-by: Michal Marek <mmarek@suse.cz>
-rw-r--r--scripts/kallsyms.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c
index e3902fb..60dd3eb 100644
--- a/scripts/kallsyms.c
+++ b/scripts/kallsyms.c
@@ -107,12 +107,8 @@ static int read_symbol(FILE *in, struct sym_entry *s)
rc = fscanf(in, "%llx %c %499s\n", &s->addr, &stype, str);
if (rc != 3) {
- if (rc != EOF) {
- /* skip line. sym is used as dummy to
- * shut of "warn_unused_result" warning.
- */
- sym = fgets(str, 500, in);
- }
+ if (rc != EOF && fgets(str, 500, in) == NULL)
+ fprintf(stderr, "Read error or end of file.\n");
return -1;
}
OpenPOWER on IntegriCloud