hw/lan9118.c: Add missing 'break' to fix buffer overrun
Add a missing 'break' statement to fix a buffer overrun when executing the EEPROM write-all command. Spotted by Coverity (see bug 887883). Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
This commit is contained in:
parent
04c5b17a74
commit
0e3b800e71
|
@ -863,6 +863,7 @@ static void lan9118_eeprom_cmd(lan9118_state *s, int cmd, int addr)
|
||||||
} else {
|
} else {
|
||||||
DPRINTF("EEPROM Write All (ignored)\n");
|
DPRINTF("EEPROM Write All (ignored)\n");
|
||||||
}
|
}
|
||||||
|
break;
|
||||||
case 5: /* ERASE */
|
case 5: /* ERASE */
|
||||||
if (s->eeprom_writable) {
|
if (s->eeprom_writable) {
|
||||||
s->eeprom[addr] = 0xff;
|
s->eeprom[addr] = 0xff;
|
||||||
|
|
Loading…
Reference in New Issue