From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 937EA2BB13 for ; Thu, 19 Mar 2026 13:52:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773928336; cv=none; b=KDNngkMOQg93dvK5ZHICwNsz78q3Zq20mi1dsTVUQZctjqBeICxdp/pE5UEqjxId2IpwhVGfBdobraZ3p9YWA9Ly57rv6Cs7NRbhF2ZPXUxEM8EFl+BMDDrKjijq0dHVqo2sWpMWl7e1UzB9j1xjZBXnLxF2bg/QxaUnh8Xmh/Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773928336; c=relaxed/simple; bh=hUcjmqFvA7mzdFV1clGlf1P3yjtUWGoenSTV4y/N3N0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=dX3PU6I8Q2SQ9fzGzOymvevBSSZey/GBFMZ+yiXjKExSj6l8rBafUEEx5GPNzgFd3P/OpH2hkT5S3tRYCV63g+/yGQ/HKFvz52wAxRKiuomZSBfvtUhd1rHq2ixqJDubc89ryvhHqi7u7GeicZKGzGQKSCARbf+wEFA7D0ibebM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=sBqNC1iH; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="sBqNC1iH" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 11F64C19424; Thu, 19 Mar 2026 13:52:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1773928336; bh=hUcjmqFvA7mzdFV1clGlf1P3yjtUWGoenSTV4y/N3N0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=sBqNC1iH/FKpVERlRzq5AP+S7EDsKZHlOZnVGK5sQWZhmXKnAnWRJP1UvYldBnmW/ 6PZblNo1EqmT0h7o+/JYbu+rSQaCHhP2QDlpqzrs8/hSvUjuByy+p6ZxP7uDATi8K8 F5+7DK8TUl1ClLFveD9VMVfAnJITlLt7cegcNc8z/456JcpT+88aIkMy/fqpjk6GJd HRioYsRWFrNCwl1xeZTQkZOPDiuTRZRFO11XcwqiOqZdMxTj28c5YAGl+WphctuDem dI9h1jYFROsTF0Pb2j2ZMic0ibBWn6YnpIAIS2AnJi5qcxT0iVA0XQSoUC4THhLbQe 1g2bSRL03SNTQ== From: Sasha Levin To: stable@vger.kernel.org Cc: "Masami Hiramatsu (Google)" , Sasha Levin Subject: [PATCH 6.6.y 1/2] kprobes: Remove unneeded goto Date: Thu, 19 Mar 2026 09:52:11 -0400 Message-ID: <20260319135212.2484493-1-sashal@kernel.org> X-Mailer: git-send-email 2.51.0 In-Reply-To: <2026031701-refueling-blob-24d8@gregkh> References: <2026031701-refueling-blob-24d8@gregkh> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: "Masami Hiramatsu (Google)" [ Upstream commit 5e5b8b49335971b68b54afeb0e7ded004945af07 ] Remove unneeded gotos. Since the labels referred by these gotos have only one reference for each, we can replace those gotos with the referred code. Link: https://lore.kernel.org/all/173371211203.480397.13988907319659165160.stgit@devnote2/ Signed-off-by: Masami Hiramatsu (Google) Stable-dep-of: 5ef268cb7a0a ("kprobes: Remove unneeded warnings from __arm_kprobe_ftrace()") Signed-off-by: Sasha Levin --- kernel/kprobes.c | 45 +++++++++++++++++++++------------------------ 1 file changed, 21 insertions(+), 24 deletions(-) diff --git a/kernel/kprobes.c b/kernel/kprobes.c index c10954bd84448..70827cbf6035d 100644 --- a/kernel/kprobes.c +++ b/kernel/kprobes.c @@ -1083,20 +1083,18 @@ static int __arm_kprobe_ftrace(struct kprobe *p, struct ftrace_ops *ops, if (*cnt == 0) { ret = register_ftrace_function(ops); - if (WARN(ret < 0, "Failed to register kprobe-ftrace (error %d)\n", ret)) - goto err_ftrace; + if (WARN(ret < 0, "Failed to register kprobe-ftrace (error %d)\n", ret)) { + /* + * At this point, sinec ops is not registered, we should be sefe from + * registering empty filter. + */ + ftrace_set_filter_ip(ops, (unsigned long)p->addr, 1, 0); + return ret; + } } (*cnt)++; return ret; - -err_ftrace: - /* - * At this point, sinec ops is not registered, we should be sefe from - * registering empty filter. - */ - ftrace_set_filter_ip(ops, (unsigned long)p->addr, 1, 0); - return ret; } static int arm_kprobe_ftrace(struct kprobe *p) @@ -1453,7 +1451,7 @@ _kprobe_addr(kprobe_opcode_t *addr, const char *symbol_name, unsigned long offset, bool *on_func_entry) { if ((symbol_name && addr) || (!symbol_name && !addr)) - goto invalid; + return ERR_PTR(-EINVAL); if (symbol_name) { /* @@ -1483,11 +1481,10 @@ _kprobe_addr(kprobe_opcode_t *addr, const char *symbol_name, * at the start of the function. */ addr = arch_adjust_kprobe_addr((unsigned long)addr, offset, on_func_entry); - if (addr) - return addr; + if (!addr) + return ERR_PTR(-EINVAL); -invalid: - return ERR_PTR(-EINVAL); + return addr; } static kprobe_opcode_t *kprobe_addr(struct kprobe *p) @@ -1510,15 +1507,15 @@ static struct kprobe *__get_valid_kprobe(struct kprobe *p) if (unlikely(!ap)) return NULL; - if (p != ap) { - list_for_each_entry(list_p, &ap->list, list) - if (list_p == p) - /* kprobe p is a valid probe */ - goto valid; - return NULL; - } -valid: - return ap; + if (p == ap) + return ap; + + list_for_each_entry(list_p, &ap->list, list) + if (list_p == p) + /* kprobe p is a valid probe */ + return ap; + + return NULL; } /* -- 2.51.0