diff options
| author | Jiri Olsa <[email protected]> | 2018-02-15 13:26:27 +0100 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2018-02-16 14:25:56 -0300 |
| commit | c53b4bb02b45ceec7a590e47820afbb5cef0bb81 (patch) | |
| tree | ffe56249f50fd1655934f65300c0f79e2ea2635b | |
| parent | f1f8ad52f8bf1239282737a2a5c3bd450300cc78 (diff) | |
tools lib symbol: Skip non-address kallsyms line
Adding check on failed attempt to parse the address and skip the line
parsing early in that case.
The address can be replaced with '(null)' string in case user don't have
enough permissions, like:
$ cat /proc/kallsyms
(null) A irq_stack_union
(null) A __per_cpu_start
...
Signed-off-by: Jiri Olsa <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
| -rw-r--r-- | tools/lib/symbol/kallsyms.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/lib/symbol/kallsyms.c b/tools/lib/symbol/kallsyms.c index 914cb8e3d40b..689b6a130dd7 100644 --- a/tools/lib/symbol/kallsyms.c +++ b/tools/lib/symbol/kallsyms.c @@ -38,6 +38,10 @@ int kallsyms__parse(const char *filename, void *arg, len = hex2u64(line, &start); + /* Skip the line if we failed to parse the address. */ + if (!len) + continue; + len++; if (len + 2 >= line_len) continue; |