Commit f360ea56 authored by Linyu Yuan's avatar Linyu Yuan Committed by Steven Rostedt (Google)
Browse files

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/1656296348-16111-3-git-send-email-quic_linyyuan@quicinc.com/



Acked-by: default avatarMasami Hiramatsu (Google) <mhiramat@kernel.org>
Reviewed-by: default avatarTom Zanussi <zanussi@kernel.org>
Signed-off-by: default avatarLinyu Yuan <quic_linyyuan@quicinc.com>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
parent b774926c

kernel/trace/trace_eprobe.c

+0 −2
Original line number Diff line number Diff line
@@ -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);