Skip to content

Commit 2431c1e

Browse files
Yuuoniygregkh
authored andcommitted
net: phy: mediatek: fix nvmem cell reference leak in mt798x_phy_calibration
commit 1e5a541 upstream. When nvmem_cell_read() fails in mt798x_phy_calibration(), the function returns without calling nvmem_cell_put(), leaking the cell reference. Move nvmem_cell_put() right after nvmem_cell_read() to ensure the cell reference is always released regardless of the read result. Found via static analysis and code review. Fixes: 98c485e ("net: phy: add driver for MediaTek SoC built-in GE PHYs") Cc: stable@vger.kernel.org Signed-off-by: Miaoqian Lin <linmq006@gmail.com> Reviewed-by: Daniel Golle <daniel@makrotopia.org> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Link: https://patch.msgid.link/20251211081313.2368460-1-linmq006@gmail.com Signed-off-by: Paolo Abeni <pabeni@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent c9a8cd3 commit 2431c1e

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/net/phy/mediatek/mtk-ge-soc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1167,9 +1167,9 @@ static int mt798x_phy_calibration(struct phy_device *phydev)
11671167
}
11681168

11691169
buf = (u32 *)nvmem_cell_read(cell, &len);
1170+
nvmem_cell_put(cell);
11701171
if (IS_ERR(buf))
11711172
return PTR_ERR(buf);
1172-
nvmem_cell_put(cell);
11731173

11741174
if (!buf[0] || !buf[1] || !buf[2] || !buf[3] || len < 4 * sizeof(u32)) {
11751175
phydev_err(phydev, "invalid efuse data\n");

0 commit comments

Comments
 (0)