diff options
author | Namhyung Kim <namhyung@kernel.org> | 2014-06-11 17:06:53 +0900 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2014-07-01 07:13:40 -0400 |
commit | 1f61be007e16a5d60b1cf868aa30d87f181e8e14 (patch) | |
tree | c35734b8355041e7bec6bf6ff9f4777abbc79fe5 /kernel | |
parent | a737e6dd7bfbd6d87ce1525840e6957bcb6e47e6 (diff) | |
download | op-kernel-dev-1f61be007e16a5d60b1cf868aa30d87f181e8e14.zip op-kernel-dev-1f61be007e16a5d60b1cf868aa30d87f181e8e14.tar.gz |
ftrace: Fix memory leak on failure path in ftrace_allocate_pages()
As struct ftrace_page is managed in a single linked list, it should
free from the start page.
Link: http://lkml.kernel.org/p/1402474014-28655-1-git-send-email-namhyung@kernel.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/ftrace.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 39df319..e14ff4c 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c @@ -2637,7 +2637,8 @@ ftrace_allocate_pages(unsigned long num_to_init) return start_pg; free_pages: - while (start_pg) { + pg = start_pg; + while (pg) { order = get_count_order(pg->size / ENTRIES_PER_PAGE); free_pages((unsigned long)pg->records, order); start_pg = pg->next; |