diff options
| author | Jean Sacren <sakiwit@gmail.com> | 2010-09-10 23:13:33 -0600 |
|---|---|---|
| committer | Michal Marek <mmarek@suse.cz> | 2010-09-29 16:18:27 +0200 |
| commit | ef894870c6c38580e540c29cfb22a827d54a950a (patch) | |
| tree | b1065bc4e2e637aa2c80a07040636b0e9a98de7e | |
| parent | 60eab877a9581b479c6c7d7913ce8070a0ace960 (diff) | |
| download | linux-ef894870c6c38580e540c29cfb22a827d54a950a.tar.gz linux-ef894870c6c38580e540c29cfb22a827d54a950a.tar.bz2 linux-ef894870c6c38580e540c29cfb22a827d54a950a.zip | |
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.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c index e3902fb39afd..60dd3eb9366e 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; } |