tracing/perf: Use strndup_user() instead of buggy open-coded version
authorJann Horn <jannh@google.com>
Wed, 20 Feb 2019 16:54:43 +0000 (17:54 +0100)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Thu, 21 Feb 2019 15:35:10 +0000 (10:35 -0500)
commit83540fbc8812a580b6ad8f93f4c29e62e417687e
treebbced260938642e388e780d28245bea212f25934
parentb689269bd1bb7b1bffb01c1d2d212e66b54f2eb2
tracing/perf: Use strndup_user() instead of buggy open-coded version

The first version of this method was missing the check for
`ret == PATH_MAX`; then such a check was added, but it didn't call kfree()
on error, so there was still a small memory leak in the error case.
Fix it by using strndup_user() instead of open-coding it.

Link: http://lkml.kernel.org/r/20190220165443.152385-1-jannh@google.com
Cc: Ingo Molnar <mingo@kernel.org>
Cc: stable@vger.kernel.org
Fixes: 0eadcc7a7bc0 ("perf/core: Fix perf_uprobe_init()")
Reviewed-by: Masami Hiramatsu <mhiramat@kernel.org>
Acked-by: Song Liu <songliubraving@fb.com>
Signed-off-by: Jann Horn <jannh@google.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
kernel/trace/trace_event_perf.c