Skip to content

Commit c283782

Browse files
jason77-wangkuba-moo
authored andcommitted
net: phy: realtek: Check the index value in led_hw_control_get
Just like rtl8211f_led_hw_is_supported() and rtl8211f_led_hw_control_set(), the rtl8211f_led_hw_control_get() also needs to check the index value, otherwise the caller is likely to get an incorrect rules. Fixes: 1778480 ("net: phy: realtek: Add support for PHY LEDs on RTL8211F") Signed-off-by: Hui Wang <[email protected]> Reviewed-by: Marek Vasut <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Jakub Kicinski <[email protected]>
1 parent aec7291 commit c283782

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

drivers/net/phy/realtek.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -527,6 +527,9 @@ static int rtl8211f_led_hw_control_get(struct phy_device *phydev, u8 index,
527527
{
528528
int val;
529529

530+
if (index >= RTL8211F_LED_COUNT)
531+
return -EINVAL;
532+
530533
val = phy_read_paged(phydev, 0xd04, RTL8211F_LEDCR);
531534
if (val < 0)
532535
return val;

0 commit comments

Comments
 (0)