From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 B6281402BBD; Wed, 20 May 2026 18:41:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779302507; cv=none; b=FfFWkZlj426I3gHWz8mgrUIMkUjn34bjyxZvc7bgE02GMI0OhlcR0WHSXrn0wtkrbB9mY4sS2feaSrCqIp7dJd5FB5MnVn1Ge+azb5soyc/TmwInr7WEImtdAegkrI6mQNEe8fU87LCRWx1B+4/jVgdmGdvNgamc8YXjps+08Uo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779302507; c=relaxed/simple; bh=FvLoH030qSga7ohX/lENRhJ/n0m154mGvD1wOkgITtA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=dWzq6+bhyX9MstqEqBl8BlTFdUZrnsXkBZNWBB+8mHMu0jpe2NQGUMkehnmgs6a7LbF4ladPb2mNaR3ogvA7LK9oNUfL+VjpWdwRBum9ncgmFxl65X9iZKokgnDnyFvpEuZnM0u/FmXdMUB8CRXToti0pAPM8Ek0NaKJ9CqhcXQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=lrZleveO; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="lrZleveO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 102C71F000E9; Wed, 20 May 2026 18:41:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1779302506; bh=6O5A2XGmqBhb+Vux+gV7gQjzl9HvV/RiPlSU0qDmtLg=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=lrZleveOOktj2O+iEVkfJLO6Fj5RWwQmoAYw6kUSi1Nasi+oS7UJJdnFHiKFVlmeE Os0WcCLLLB4FBzNLhofJ244HhkFpyQT/d/5VgPYbmw78vp0OdrL43UzHtEaeeUjiWT TnxJLj2QYYh1kVdwY2iCe9sMXOEXvePnWN4ZeZnA= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Brian Masney , =?UTF-8?q?Beno=C3=AEt=20Monin?= , Nobuhiro Iwamatsu , Stephen Boyd , Sasha Levin Subject: [PATCH 6.6 292/508] clk: visconti: pll: initialize clk_init_data to zero Date: Wed, 20 May 2026 18:21:55 +0200 Message-ID: <20260520162104.972343249@linuxfoundation.org> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260520162058.573354582@linuxfoundation.org> References: <20260520162058.573354582@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 6.6-stable review patch. If anyone has any objections, please let me know. ------------------ From: Brian Masney [ Upstream commit 1603cbb64173a0e9fa7500f2a686f4aa011c58b9 ] Sashiko reported the following: > The struct clk_init_data init is declared on the stack without being > fully zero-initialized. While fields like name, flags, parent_names, > num_parents, and ops are explicitly assigned, the parent_data and > parent_hws fields are left containing stack garbage. clk_core_populate_parent_map() currently prefers the parent names over the parent data and hws, so this isn't a problem at the moment. If that ordering ever changed in the future, then this could lead to some unexpected crashes. Let's just go ahead and make sure that the struct clk_init_data is initialized to zero as a good practice. Fixes: b4cbe606dc367 ("clk: visconti: Add support common clock driver and reset driver") Link: https://sashiko.dev/#/patchset/20260326042317.122536-1-rosenp%40gmail.com Signed-off-by: Brian Masney Reviewed-by: Benoît Monin Reviewed-by: Nobuhiro Iwamatsu Signed-off-by: Stephen Boyd Signed-off-by: Sasha Levin --- drivers/clk/visconti/pll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/visconti/pll.c b/drivers/clk/visconti/pll.c index e9cd80e085dc3..a540936196ca3 100644 --- a/drivers/clk/visconti/pll.c +++ b/drivers/clk/visconti/pll.c @@ -244,7 +244,7 @@ static struct clk_hw *visconti_register_pll(struct visconti_pll_provider *ctx, const struct visconti_pll_rate_table *rate_table, spinlock_t *lock) { - struct clk_init_data init; + struct clk_init_data init = {}; struct visconti_pll *pll; struct clk_hw *pll_hw_clk; size_t len; -- 2.53.0