Messages in this thread | | | Date | Wed, 25 Feb 2015 00:12:31 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 09/11] tools lib traceevent: Add alias field to struct format_field |
| |
On Tue, Feb 24, 2015 at 05:54:58PM -0500, Steven Rostedt wrote: > On Fri, Feb 20, 2015 at 11:17:06PM +0100, Jiri Olsa wrote: > > --- > > tools/lib/traceevent/event-parse.c | 1 + > > tools/lib/traceevent/event-parse.h | 1 + > > 2 files changed, 2 insertions(+) > > > > diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c > > index afe20ed9fac8..b35664cc1edc 100644 > > --- a/tools/lib/traceevent/event-parse.c > > +++ b/tools/lib/traceevent/event-parse.c > > @@ -6236,6 +6236,7 @@ static void free_format_fields(struct format_field *field) > > next = field->next; > > free(field->type); > > free(field->name); > > + free(field->alias); > > Hmm, didn't we say that if there's no need for the alias that the alias will > simply be a pointer to name? If that's the case we need: > > if (field->alias != field->name) > free(field->alias); > free(field->name); > > Otherwise it will be a double free.
i used it separately.. like if there's no alias it's set to NULL, but I'll add the logic you described and repost, it seems ok
jirka
| |