The driver supports a fair amount of Wake-on-LAN modes, but is not
checking that the user specified one that is supported.

Fixes: 55d7de9de6c3 ("Microchip's LAN7800 family USB 2/3 to 10/100/1000 
Ethernet device driver")
Signed-off-by: Florian Fainelli <f.faine...@gmail.com>
---
 drivers/net/usb/lan78xx.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c
index a9991c5f4736..e0a588d1b8e6 100644
--- a/drivers/net/usb/lan78xx.c
+++ b/drivers/net/usb/lan78xx.c
@@ -1415,6 +1415,9 @@ static int lan78xx_set_wol(struct net_device *netdev,
        if (wol->wolopts & WAKE_ARP)
                pdata->wol |= WAKE_ARP;
 
+       if (pdata->wol == 0)
+               return -EINVAL;
+
        device_set_wakeup_enable(&dev->udev->dev, (bool)wol->wolopts);
 
        phy_ethtool_set_wol(netdev->phydev, wol);
-- 
2.17.1

Reply via email to