On Fri, Jan 30, 2015 at 09:17:19AM -0500, Steven Rostedt wrote: > On Mon, 26 Jan 2015 18:38:23 +0800 > Wang Nan <[email protected]> wrote: > > > > diff --git a/tools/lib/traceevent/event-parse.c > > b/tools/lib/traceevent/event-parse.c > > index cf3a44b..5f76003 100644 > > --- a/tools/lib/traceevent/event-parse.c > > +++ b/tools/lib/traceevent/event-parse.c > > @@ -5909,6 +5909,8 @@ static void free_format_fields(struct format_field > > *field) > > free(field->type); > > free(field->name); > > free(field); > > + if (field->destroy_priv) > > + field->destroy_priv(field); > > I think you want to call field->destroy_priv() *before* you free field.
argh.. missed that :-\ will fix thanks, jirka > > -- Steve > > > field = next; > > } > > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
