diff options
author | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-12-10 23:58:01 -0500 |
---|---|---|
committer | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-12-11 13:50:28 -0500 |
commit | 2840f84f74035e5a535959d5f17269c69fa6edc5 (patch) | |
tree | d34045c522b96a57bd2ca29c089cf6df8d9d6b6e /kernel/trace/ftrace.c | |
parent | 3cec638b3d793b7cacdec5b8072364b41caeb0e1 (diff) |
tracing: Fix memory leak of instance function hash filters
The following commands will cause a memory leak:
# cd /sys/kernel/tracing
# mkdir instances/foo
# echo schedule > instance/foo/set_ftrace_filter
# rmdir instances/foo
The reason is that the hashes that hold the filters to set_ftrace_filter and
set_ftrace_notrace are not freed if they contain any data on the instance
and the instance is removed.
Found by kmemleak detector.
Cc: stable@vger.kernel.org
Fixes: 591dffdade9f ("ftrace: Allow for function tracing instance to filter functions")
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/ftrace.c')
-rw-r--r-- | kernel/trace/ftrace.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 77734451cb05..e23eb9fc77aa 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -5460,6 +5460,7 @@ void ftrace_destroy_filter_files(struct ftrace_ops *ops) | |||
5460 | if (ops->flags & FTRACE_OPS_FL_ENABLED) | 5460 | if (ops->flags & FTRACE_OPS_FL_ENABLED) |
5461 | ftrace_shutdown(ops, 0); | 5461 | ftrace_shutdown(ops, 0); |
5462 | ops->flags |= FTRACE_OPS_FL_DELETED; | 5462 | ops->flags |= FTRACE_OPS_FL_DELETED; |
5463 | ftrace_free_filter(ops); | ||
5463 | mutex_unlock(&ftrace_lock); | 5464 | mutex_unlock(&ftrace_lock); |
5464 | } | 5465 | } |
5465 | 5466 | ||