diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 15:11:03 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 15:22:35 -0500 |
commit | 5eae7d842510d33d4410c062280eda6c935403dd (patch) | |
tree | 3761779adda8056d41ac3d203e2a9be1194d9db8 | |
parent | 9ef6839bcce7ca944c1ace4a7247cf13ca92a28f (diff) |
perf symbols: dso->name is an array, no need to check it against NULL
As it will always evaluate to 'true', as reported by clang:
util/map.c:390:36: error: address of array 'map->dso->name' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
if (map && map->dso && (map->dso->name || map->dso->long_name)) {
~~~~~~~~~~^~~~ ~~
util/map.c:393:22: error: address of array 'map->dso->name' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
else if (map->dso->name)
~~ ~~~~~~~~~~^~~~
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-x8cu007cly40kfp8xnpi9kya@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/util/map.c | 4 | ||||
-rw-r--r-- | tools/perf/util/scripting-engines/trace-event-perl.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/tools/perf/util/map.c b/tools/perf/util/map.c index 4f9a71c63026..0a943e7b1ea7 100644 --- a/tools/perf/util/map.c +++ b/tools/perf/util/map.c | |||
@@ -387,10 +387,10 @@ size_t map__fprintf_dsoname(struct map *map, FILE *fp) | |||
387 | { | 387 | { |
388 | const char *dsoname = "[unknown]"; | 388 | const char *dsoname = "[unknown]"; |
389 | 389 | ||
390 | if (map && map->dso && (map->dso->name || map->dso->long_name)) { | 390 | if (map && map->dso) { |
391 | if (symbol_conf.show_kernel_path && map->dso->long_name) | 391 | if (symbol_conf.show_kernel_path && map->dso->long_name) |
392 | dsoname = map->dso->long_name; | 392 | dsoname = map->dso->long_name; |
393 | else if (map->dso->name) | 393 | else |
394 | dsoname = map->dso->name; | 394 | dsoname = map->dso->name; |
395 | } | 395 | } |
396 | 396 | ||
diff --git a/tools/perf/util/scripting-engines/trace-event-perl.c b/tools/perf/util/scripting-engines/trace-event-perl.c index 014ecd6f67c4..c1555fd0035a 100644 --- a/tools/perf/util/scripting-engines/trace-event-perl.c +++ b/tools/perf/util/scripting-engines/trace-event-perl.c | |||
@@ -309,10 +309,10 @@ static SV *perl_process_callchain(struct perf_sample *sample, | |||
309 | if (node->map) { | 309 | if (node->map) { |
310 | struct map *map = node->map; | 310 | struct map *map = node->map; |
311 | const char *dsoname = "[unknown]"; | 311 | const char *dsoname = "[unknown]"; |
312 | if (map && map->dso && (map->dso->name || map->dso->long_name)) { | 312 | if (map && map->dso) { |
313 | if (symbol_conf.show_kernel_path && map->dso->long_name) | 313 | if (symbol_conf.show_kernel_path && map->dso->long_name) |
314 | dsoname = map->dso->long_name; | 314 | dsoname = map->dso->long_name; |
315 | else if (map->dso->name) | 315 | else |
316 | dsoname = map->dso->name; | 316 | dsoname = map->dso->name; |
317 | } | 317 | } |
318 | if (!hv_stores(elem, "dso", newSVpv(dsoname,0))) { | 318 | if (!hv_stores(elem, "dso", newSVpv(dsoname,0))) { |