mirror of
				https://git.openwrt.org/openwrt/openwrt.git
				synced 2025-10-31 16:38:20 +00:00 
			
		
		
		
	This is an automatically generated commit which aids following Kernel patch history, as git will see the move and copy as a rename thus defeating the purpose. For the original discussion see: https://lists.openwrt.org/pipermail/openwrt-devel/2023-October/041673.html Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com> Link: https://github.com/openwrt/openwrt/pull/16547 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
		
			
				
	
	
		
			33 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			33 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
| From c283782fc5d60c4d8169137c6f955aa3553d3b3d Mon Sep 17 00:00:00 2001
 | |
| From: Hui Wang <hui.wang@canonical.com>
 | |
| Date: Fri, 27 Sep 2024 19:46:10 +0800
 | |
| Subject: [PATCH] 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: 17784801d888 ("net: phy: realtek: Add support for PHY LEDs on RTL8211F")
 | |
| Signed-off-by: Hui Wang <hui.wang@canonical.com>
 | |
| Reviewed-by: Marek Vasut <marex@denx.de>
 | |
| Link: https://patch.msgid.link/20240927114610.1278935-1-hui.wang@canonical.com
 | |
| Signed-off-by: Jakub Kicinski <kuba@kernel.org>
 | |
| ---
 | |
|  drivers/net/phy/realtek.c | 3 +++
 | |
|  1 file changed, 3 insertions(+)
 | |
| 
 | |
| --- a/drivers/net/phy/realtek.c
 | |
| +++ b/drivers/net/phy/realtek.c
 | |
| @@ -527,6 +527,9 @@ static int rtl8211f_led_hw_control_get(s
 | |
|  {
 | |
|  	int val;
 | |
|  
 | |
| +	if (index >= RTL8211F_LED_COUNT)
 | |
| +		return -EINVAL;
 | |
| +
 | |
|  	val = phy_read_paged(phydev, 0xd04, RTL8211F_LEDCR);
 | |
|  	if (val < 0)
 | |
|  		return val;
 |