]> git.ipfire.org Git - thirdparty/linux.git/commit
clk: rs9: Reserve 8 struct clk_hw slots for for 9FGV0841
authorMarek Vasut <marek.vasut+renesas@mailbox.org>
Wed, 21 Jan 2026 23:26:38 +0000 (00:26 +0100)
committerStephen Boyd <sboyd@kernel.org>
Tue, 3 Feb 2026 00:35:18 +0000 (16:35 -0800)
commit5ec820fc28d0b8a0f3890d476b1976f20e8343cc
tree1247dc9d45b2a49107a7d09924a67117c689648c
parent83436f2bf4e0cd84932a8fe5bce33af3b9e58120
clk: rs9: Reserve 8 struct clk_hw slots for for 9FGV0841

The 9FGV0841 has 8 outputs and registers 8 struct clk_hw, make sure
there are 8 slots for those newly registered clk_hw pointers, else
there is going to be out of bounds write when pointers 4..7 are set
into struct rs9_driver_data .clk_dif[4..7] field.

Since there are other structure members past this struct clk_hw
pointer array, writing to .clk_dif[4..7] fields corrupts both
the struct rs9_driver_data content and data around it, sometimes
without crashing the kernel. However, the kernel does surely
crash when the driver is unbound or during suspend.

Fix this, increase the struct clk_hw pointer array size to the
maximum output count of 9FGV0841, which is the biggest chip that
is supported by this driver.

Cc: stable@vger.kernel.org
Fixes: f0e5e1800204 ("clk: rs9: Add support for 9FGV0841")
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Tested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Closes: https://lore.kernel.org/CAMuHMdVyQpOBT+Ho+mXY07fndFN9bKJdaaWGn91WOFnnYErLyg@mail.gmail.com
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk-renesas-pcie.c