projects
/
openwrt
/
staging
/
blogic.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
ab0cce5
)
perf evlist: Explicititely initialize input_name
author
Arnaldo Carvalho de Melo
<
[email protected]
>
Thu, 24 May 2012 00:47:51 +0000
(21:47 -0300)
committer
Arnaldo Carvalho de Melo
<
[email protected]
>
Thu, 24 May 2012 00:47:51 +0000
(21:47 -0300)
It was a global variable, so it was initialized, implicitely, to zero by
being placed in the bss.
Now it is just a local variable that is then passed to the __cmd_evlist
routine, so it must be explicitely set to NULL.
The problem manifested on a Fedora 17 system, using:
gcc version 4.7.0
20120507
(Red Hat 4.7.0-5) (GCC)
But not on several other systems, by luck.
Reported-by: Ingo Molnar <
[email protected]
>
Cc: David Ahern <
[email protected]
>
Cc: Frederic Weisbecker <
[email protected]
>
Cc: Mike Galbraith <
[email protected]
>
Cc: Namhyung Kim <
[email protected]
>
Cc: Paul Mackerras <
[email protected]
>
Cc: Peter Zijlstra <
[email protected]
>
Cc: Stephane Eranian <
[email protected]
>
Link:
http://lkml.kernel.org/n/
[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <
[email protected]
>
tools/perf/builtin-evlist.c
patch
|
blob
|
history
diff --git
a/tools/perf/builtin-evlist.c
b/tools/perf/builtin-evlist.c
index e52d77ec7084e02e4fa318e873ff2cf1679a222d..acd78dc283411692f9e7e48b78085d9b5da63471 100644
(file)
--- a/
tools/perf/builtin-evlist.c
+++ b/
tools/perf/builtin-evlist.c
@@
-116,7
+116,7
@@
static const char * const evlist_usage[] = {
int cmd_evlist(int argc, const char **argv, const char *prefix __used)
{
struct perf_attr_details details = { .verbose = false, };
- const char *input_name;
+ const char *input_name
= NULL
;
const struct option options[] = {
OPT_STRING('i', "input", &input_name, "file",
"Input file name"),