diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-11-24 21:49:33 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-11-24 21:50:52 -0500 |
| commit | 426d581e89e3cbec5c353ceaedcaf5686392beb3 (patch) | |
| tree | 6d73a54488ffe05ff475be32790e6ddaf7c492ae | |
| parent | 7596e23373165960e244a6cf212495cd1d6b7ac4 (diff) | |
Move copying of system name higher in event initialization
Even events that fail should still have a system name. Otherwise
referencing an event without it will segfault.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
| -rw-r--r-- | parse-events.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/parse-events.c b/parse-events.c index 87950e2..54a6ab0 100644 --- a/parse-events.c +++ b/parse-events.c | |||
| @@ -3459,6 +3459,8 @@ int pevent_parse_event(char *buf, unsigned long size, char *sys) | |||
| 3459 | if (event->id < 0) | 3459 | if (event->id < 0) |
| 3460 | die("failed to read event id"); | 3460 | die("failed to read event id"); |
| 3461 | 3461 | ||
| 3462 | event->system = strdup(sys); | ||
| 3463 | |||
| 3462 | ret = event_read_format(event); | 3464 | ret = event_read_format(event); |
| 3463 | if (ret < 0) { | 3465 | if (ret < 0) { |
| 3464 | warning("failed to read event format for %s", event->name); | 3466 | warning("failed to read event format for %s", event->name); |
| @@ -3471,8 +3473,6 @@ int pevent_parse_event(char *buf, unsigned long size, char *sys) | |||
| 3471 | goto event_failed; | 3473 | goto event_failed; |
| 3472 | } | 3474 | } |
| 3473 | 3475 | ||
| 3474 | event->system = strdup(sys); | ||
| 3475 | |||
| 3476 | add_event(event); | 3476 | add_event(event); |
| 3477 | 3477 | ||
| 3478 | if (!ret && (event->flags & EVENT_FL_ISFTRACE)) { | 3478 | if (!ret && (event->flags & EVENT_FL_ISFTRACE)) { |
