diff options
| author | Namhyung Kim <[email protected]> | 2012-04-23 13:58:34 +0900 |
|---|---|---|
| committer | Namhyung Kim <[email protected]> | 2012-07-04 13:40:31 +0900 |
| commit | 4b5632bc3122ec9b7e875294c8abe92f1573196a (patch) | |
| tree | 256ec7c8759be0e5805bc7bc542b15ebdb29894f | |
| parent | 3831a42deba59bf26618b0dd6fa1320a0a94ebd9 (diff) | |
tools lib traceevent: Do not call add_event() again if allocation failed
When memory allocation for the field name is failed, do not goto
event_failed since we added the event already.
Signed-off-by: Namhyung Kim <[email protected]>
Cc: Frederic Weisbecker <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Borislav Petkov <[email protected]>
Cc: David Ahern <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Steven Rostedt <[email protected]>
Signed-off-by: Namhyung Kim <[email protected]>
| -rw-r--r-- | tools/lib/traceevent/event-parse.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c index 337ca02fb525..99b0cd4b79d4 100644 --- a/tools/lib/traceevent/event-parse.c +++ b/tools/lib/traceevent/event-parse.c @@ -4770,7 +4770,8 @@ int pevent_parse_event(struct pevent *pevent, arg->field.name = strdup(field->name); if (!arg->field.name) { do_warning("failed to allocate field name"); - goto event_failed; + event->flags |= EVENT_FL_FAILED; + return -1; } arg->field.field = field; } |