Skip to content

Commit 7c14a4e

Browse files
inguinSasha Levin
authored and
Sasha Levin
committed
net: phy: dp83869: fix memory corruption when enabling fiber
[ Upstream commit a842e44 ] When configuring the fiber port, the DP83869 PHY driver incorrectly calls linkmode_set_bit() with a bit mask (1 << 10) rather than a bit number (10). This corrupts some other memory location -- in case of arm64 the priv pointer in the same structure. Since the advertising flags are updated from supported at the end of the function the incorrect line isn't needed at all and can be removed. Fixes: a29de52 ("net: dp83869: Add ability to advertise Fiber connection") Signed-off-by: Ingo van Lil <[email protected]> Reviewed-by: Alexander Sverdlin <[email protected]> Reviewed-by: Andrew Lunn <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Jakub Kicinski <[email protected]> Signed-off-by: Sasha Levin <[email protected]>
1 parent 37163d1 commit 7c14a4e

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

drivers/net/phy/dp83869.c

-1
Original file line numberDiff line numberDiff line change
@@ -644,7 +644,6 @@ static int dp83869_configure_fiber(struct phy_device *phydev,
644644
phydev->supported);
645645

646646
linkmode_set_bit(ETHTOOL_LINK_MODE_FIBRE_BIT, phydev->supported);
647-
linkmode_set_bit(ADVERTISED_FIBRE, phydev->advertising);
648647

649648
if (dp83869->mode == DP83869_RGMII_1000_BASE) {
650649
linkmode_set_bit(ETHTOOL_LINK_MODE_1000baseX_Full_BIT,

0 commit comments

Comments
 (0)