diff options
| author | Xiu Jianfeng <[email protected]> | 2022-11-15 09:44:45 +0800 |
|---|---|---|
| committer | Steven Rostedt (Google) <[email protected]> | 2022-11-22 18:09:50 -0500 |
| commit | ccc6e5900745a60073e4967f04b618cdd92b63d6 (patch) | |
| tree | 3e48612d0fef0ff4bb8bb712f5a4dc188c4b437c | |
| parent | 0a068f4a717f2a68b34452de682accbb1a40bed0 (diff) | |
tracing/user_events: Fix memory leak in user_event_create()
Before current_user_event_group(), it has allocated memory and save it
in @name, this should freed before return error.
Link: https://lkml.kernel.org/r/[email protected]
Fixes: e5d271812e7a ("tracing/user_events: Move pages/locks into groups to prepare for namespaces")
Signed-off-by: Xiu Jianfeng <[email protected]>
Acked-by: Masami Hiramatsu (Google) <[email protected]>
Acked-by: Beau Belgrave <[email protected]>
Signed-off-by: Steven Rostedt (Google) <[email protected]>
| -rw-r--r-- | kernel/trace/trace_events_user.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c index ae78c2d53c8a..539b08ae7020 100644 --- a/kernel/trace/trace_events_user.c +++ b/kernel/trace/trace_events_user.c @@ -1100,8 +1100,10 @@ static int user_event_create(const char *raw_command) group = current_user_event_group(); - if (!group) + if (!group) { + kfree(name); return -ENOENT; + } mutex_lock(&group->reg_mutex); |