diff options
| author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-03-24 09:57:57 -0400 |
|---|---|---|
| committer | Sasha Levin <sasha.levin@oracle.com> | 2016-05-17 22:05:14 -0400 |
| commit | 1b57d360ad96e2e54ddbffcb391527702a1ae86f (patch) | |
| tree | 3098122a876489907a6998f8b99e35afbc4d6047 | |
| parent | f6c3f874c62a34758e1532d20e97ab8ea96f7106 (diff) | |
| download | lwn-1b57d360ad96e2e54ddbffcb391527702a1ae86f.tar.gz lwn-1b57d360ad96e2e54ddbffcb391527702a1ae86f.zip | |
tools lib traceevent: Free filter tokens in process_filter()
[ Upstream commit e1644aae4589274223c1ab9072ddbda98dd97f6a ]
valgrind showed that the filter token wasn't being freed properly in
process_filter().
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/20150324135923.817723903@goodmis.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
| -rw-r--r-- | tools/lib/traceevent/parse-filter.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/lib/traceevent/parse-filter.c b/tools/lib/traceevent/parse-filter.c index b50234402fc2..0144b3d1bb77 100644 --- a/tools/lib/traceevent/parse-filter.c +++ b/tools/lib/traceevent/parse-filter.c @@ -1058,6 +1058,7 @@ process_filter(struct event_format *event, struct filter_arg **parg, *parg = current_op; else *parg = current_exp; + free(token); return PEVENT_ERRNO__UNBALANCED_PAREN; } break; @@ -1168,6 +1169,7 @@ process_filter(struct event_format *event, struct filter_arg **parg, *parg = current_op; + free(token); return 0; fail_alloc: |