aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLinyu Yuan <[email protected]>2022-06-27 10:19:06 +0800
committerSteven Rostedt (Google) <[email protected]>2022-07-24 19:11:17 -0400
commitf360ea5641dc9473ad485e882c8ac3b1aa2672ff (patch)
treea2a9e7c5fffe603e9b853a49613509aa940233b4
parentb774926c733850037b15c50f893383aa71bd8695 (diff)
tracing: eprobe: Remove duplicate is_good_name() operation
traceprobe_parse_event_name() already validate SYSTEM and EVENT name, there is no need to call is_good_name() after it. Link: https://lore.kernel.org/all/[email protected]/ Acked-by: Masami Hiramatsu (Google) <[email protected]> Reviewed-by: Tom Zanussi <[email protected]> Signed-off-by: Linyu Yuan <[email protected]> Signed-off-by: Steven Rostedt (Google) <[email protected]>
-rw-r--r--kernel/trace/trace_eprobe.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/trace/trace_eprobe.c b/kernel/trace/trace_eprobe.c
index b805b570305f..8979cb9ec37a 100644
--- a/kernel/trace/trace_eprobe.c
+++ b/kernel/trace/trace_eprobe.c
@@ -887,8 +887,6 @@ static int __trace_eprobe_create(int argc, const char *argv[])
sys_event - argv[1]);
if (ret || !sys_name)
goto parse_error;
- if (!is_good_name(sys_event) || !is_good_name(sys_name))
- goto parse_error;
mutex_lock(&event_mutex);
event_call = find_and_get_event(sys_name, sys_event);