From: "Steven Rostedt (Red Hat)" <[email protected]>

valgrind showed that the filter token wasn't being freed properly
in process_filter().

Signed-off-by: Steven Rostedt <[email protected]>
---
 tools/lib/traceevent/parse-filter.c | 2 ++
 1 file changed, 2 insertions(+)

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:
-- 
2.1.4


--
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/

Reply via email to