Commit 61217be8 authored by zhangyue's avatar zhangyue Committed by David S. Miller

net: tulip: de4x5: fix the problem that the array 'lp->phy[8]' may be out of bound

In line 5001, if all id in the array 'lp->phy[8]' is not 0, when the
'for' end, the 'k' is 8.

At this time, the array 'lp->phy[8]' may be out of bound.
Signed-off-by: default avatarzhangyue <zhangyue1@kylinos.cn>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4e5d2124
...@@ -5000,19 +5000,23 @@ mii_get_phy(struct net_device *dev) ...@@ -5000,19 +5000,23 @@ mii_get_phy(struct net_device *dev)
} }
if ((j == limit) && (i < DE4X5_MAX_MII)) { if ((j == limit) && (i < DE4X5_MAX_MII)) {
for (k=0; k < DE4X5_MAX_PHY && lp->phy[k].id; k++); for (k=0; k < DE4X5_MAX_PHY && lp->phy[k].id; k++);
lp->phy[k].addr = i; if (k < DE4X5_MAX_PHY) {
lp->phy[k].id = id; lp->phy[k].addr = i;
lp->phy[k].spd.reg = GENERIC_REG; /* ANLPA register */ lp->phy[k].id = id;
lp->phy[k].spd.mask = GENERIC_MASK; /* 100Mb/s technologies */ lp->phy[k].spd.reg = GENERIC_REG; /* ANLPA register */
lp->phy[k].spd.value = GENERIC_VALUE; /* TX & T4, H/F Duplex */ lp->phy[k].spd.mask = GENERIC_MASK; /* 100Mb/s technologies */
lp->mii_cnt++; lp->phy[k].spd.value = GENERIC_VALUE; /* TX & T4, H/F Duplex */
lp->active++; lp->mii_cnt++;
printk("%s: Using generic MII device control. If the board doesn't operate,\nplease mail the following dump to the author:\n", dev->name); lp->active++;
j = de4x5_debug; printk("%s: Using generic MII device control. If the board doesn't operate,\nplease mail the following dump to the author:\n", dev->name);
de4x5_debug |= DEBUG_MII; j = de4x5_debug;
de4x5_dbg_mii(dev, k); de4x5_debug |= DEBUG_MII;
de4x5_debug = j; de4x5_dbg_mii(dev, k);
printk("\n"); de4x5_debug = j;
printk("\n");
} else {
goto purgatory;
}
} }
} }
purgatory: purgatory:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment