diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-07-05 07:39:18 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-07-05 10:30:21 +0200 |
commit | be9038859e56f729cc9d3b070a35fb8829a73696 (patch) | |
tree | e258cec23a938ba1d1a9a38119420171c52f31c0 /kernel/time.c | |
parent | 91b4eaea93f5be95f4477554399680a53aff2343 (diff) |
perf report: Use a modifiable string for default callchain options
If the user doesn't provide options to tune his callchain output
(ie: if he uses -c without arguments) then the default value passed
in the OPT_CALLBACK_DEFAULT() macro is used.
But it's parsed later by strtok() which will replace comma separators
to a zero. This may segfault as we are using a read-only string.
Use a modifiable one instead, and also fix the "100%" default
minimum threshold value by turning it into a 0 (output every callchains)
as it was intended in the origin.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Anton Blanchard <anton@samba.org>
Cc: Jens Axboe <jens.axboe@oracle.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
LKML-Reference: <1246772361-9960-2-git-send-email-fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/time.c')
0 files changed, 0 insertions, 0 deletions