phy: Fix u-boot coruption when fixed-phy is used
authorMichal Simek <michal.simek@xilinx.com>
Wed, 19 Dec 2018 15:57:38 +0000 (16:57 +0100)
committerTom Rini <trini@konsulko.com>
Thu, 27 Dec 2018 02:35:52 +0000 (21:35 -0500)
When fixed-link phy is used subnode offset is used as phy address. This
number is bigger then space allocated for bus structure (allocated via
mdio_alloc).
bus->phymap[] array has PHY_MAX_ADDR size (32).
That's why writing bus->phymap[addr] where addr is < 0 or > PHY_MAX_ADDR
is causing write to memory which can caused full U-Boot crash.

The patch is checking if address is in correct range.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
drivers/net/phy/phy.c

index e837eb7..cda4caa 100644 (file)
@@ -656,7 +656,8 @@ static struct phy_device *phy_device_create(struct mii_dev *bus, int addr,
 
        phy_probe(dev);
 
-       bus->phymap[addr] = dev;
+       if (addr >= 0 && addr < PHY_MAX_ADDR)
+               bus->phymap[addr] = dev;
 
        return dev;
 }