aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf
diff options
context:
space:
mode:
authorSteven Rostedt <srostedt@redhat.com>2009-10-14 15:43:37 -0400
committerIngo Molnar <mingo@elte.hu>2009-10-15 04:42:37 -0400
commitf1d1feecf07261d083859ecfef0d4399036f9683 (patch)
treeaebca45d8bc2f4207b52f07ce12d45dd0561e37a /tools/perf
parentb99af874829cba2b30d212bc6fd31b56275ee4d2 (diff)
perf tools: Handle newlines in trace parsing better
New lines between args in the trace format can break the parsing. This should not be the case. Signed-off-by: Steven Rostedt <srostedt@redhat.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> LKML-Reference: <20091014194358.637991808@goodmis.org> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf')
-rw-r--r--tools/perf/util/trace-event-parse.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/tools/perf/util/trace-event-parse.c b/tools/perf/util/trace-event-parse.c
index f73ee55b51e8..59e4e4db7438 100644
--- a/tools/perf/util/trace-event-parse.c
+++ b/tools/perf/util/trace-event-parse.c
@@ -1716,12 +1716,18 @@ process_arg_token(struct event *event, struct print_arg *arg,
1716 1716
1717static int event_read_print_args(struct event *event, struct print_arg **list) 1717static int event_read_print_args(struct event *event, struct print_arg **list)
1718{ 1718{
1719 enum event_type type; 1719 enum event_type type = EVENT_ERROR;
1720 struct print_arg *arg; 1720 struct print_arg *arg;
1721 char *token; 1721 char *token;
1722 int args = 0; 1722 int args = 0;
1723 1723
1724 do { 1724 do {
1725 if (type == EVENT_NEWLINE) {
1726 free_token(token);
1727 type = read_token_item(&token);
1728 continue;
1729 }
1730
1725 arg = malloc_or_die(sizeof(*arg)); 1731 arg = malloc_or_die(sizeof(*arg));
1726 memset(arg, 0, sizeof(*arg)); 1732 memset(arg, 0, sizeof(*arg));
1727 1733