diff options
| author | Tom Zanussi <tzanussi@gmail.com> | 2009-03-24 03:14:11 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-03-24 03:26:51 -0400 |
| commit | 09f1f245c79585383de63e3ca54d0f91824bff3a (patch) | |
| tree | bc3aaadf4cc0c0cd90bafd4a61ad4c15392da303 | |
| parent | 1fc2d5c11918082536acf261ce6abb1f5511053f (diff) | |
tracing/filters: free pred when clearing filters
Impact: fix (small) per trace filter modification memory leak
Free the current pred when clearing the filters via the filter files.
Signed-off-by: Tom Zanussi <tzanussi@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: =?ISO-8859-1?Q?Fr=E9d=E9ric?= Weisbecker <fweisbec@gmail.com>
LKML-Reference: <1237878851.8339.58.camel@charm-linux>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
| -rw-r--r-- | kernel/trace/trace_events.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index fdab30d6c835..a9381384aa9e 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c | |||
| @@ -516,6 +516,7 @@ event_filter_write(struct file *filp, const char __user *ubuf, size_t cnt, | |||
| 516 | 516 | ||
| 517 | if (pred->clear) { | 517 | if (pred->clear) { |
| 518 | filter_free_preds(call); | 518 | filter_free_preds(call); |
| 519 | filter_free_pred(pred); | ||
| 519 | return cnt; | 520 | return cnt; |
| 520 | } | 521 | } |
| 521 | 522 | ||
| @@ -581,6 +582,7 @@ subsystem_filter_write(struct file *filp, const char __user *ubuf, size_t cnt, | |||
| 581 | 582 | ||
| 582 | if (pred->clear) { | 583 | if (pred->clear) { |
| 583 | filter_free_subsystem_preds(system); | 584 | filter_free_subsystem_preds(system); |
| 585 | filter_free_pred(pred); | ||
| 584 | return cnt; | 586 | return cnt; |
| 585 | } | 587 | } |
| 586 | 588 | ||
