Messages in this thread | | | Date | Fri, 30 Jan 2015 09:17:19 -0500 | From | Steven Rostedt <> | Subject | Re: [PATCH v2 1/3] tools lib traceevent: add priv field to struct format_field. |
| |
On Mon, 26 Jan 2015 18:38:23 +0800 Wang Nan <wangnan0@huawei.com> 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.
-- Steve
> field = next; > } > }
| |