diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-05-26 23:04:48 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2009-06-01 19:18:03 -0400 |
commit | e16228069083a2f6b94383ac5739aea7a0f38ce4 (patch) | |
tree | 97bf6074fff3ea29daa732f92d29472cf803165a /kernel/trace/trace_stat.c | |
parent | b3dd7ba7d862707800c7ac45068f14ade2b65155 (diff) |
tracing/stat: remember to free root node
When closing a trace_stat file, we destroy the rbtree constructed during
file open, but there is memory leak that the root node is not freed.
[ Impact: fix memory leak when closing a trace_stat file ]
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Diffstat (limited to 'kernel/trace/trace_stat.c')
-rw-r--r-- | kernel/trace/trace_stat.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace_stat.c b/kernel/trace/trace_stat.c index 5816d1aebcc9..8030ec98dbad 100644 --- a/kernel/trace/trace_stat.c +++ b/kernel/trace/trace_stat.c | |||
@@ -60,8 +60,8 @@ static struct rb_node *release_next(struct rb_node *node) | |||
60 | return node->rb_right; | 60 | return node->rb_right; |
61 | else { | 61 | else { |
62 | if (!parent) | 62 | if (!parent) |
63 | return NULL; | 63 | ; |
64 | if (parent->rb_left == node) | 64 | else if (parent->rb_left == node) |
65 | parent->rb_left = NULL; | 65 | parent->rb_left = NULL; |
66 | else | 66 | else |
67 | parent->rb_right = NULL; | 67 | parent->rb_right = NULL; |