[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 19/19] i2c: Verify that the count passed in to smbus_
From: |
minyard |
Subject: |
[Qemu-devel] [PULL 19/19] i2c: Verify that the count passed in to smbus_eeprom_init() is valid |
Date: |
Thu, 28 Feb 2019 12:17:10 -0600 |
From: Corey Minyard <address@hidden>
Keep someone from passing in a bogus number
Signed-off-by: Corey Minyard <address@hidden>
---
hw/i2c/smbus_eeprom.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
index 0ba5763fc2..37167e7244 100644
--- a/hw/i2c/smbus_eeprom.c
+++ b/hw/i2c/smbus_eeprom.c
@@ -180,6 +180,8 @@ void smbus_eeprom_init(I2CBus *smbus, int nb_eeprom,
{
int i;
/* XXX: make this persistent */
+
+ assert(nb_eeprom <= 8);
uint8_t *eeprom_buf = g_malloc0(8 * SMBUS_EEPROM_SIZE);
if (eeprom_spd_size > 0) {
memcpy(eeprom_buf, eeprom_spd, eeprom_spd_size);
--
2.17.1
- [Qemu-devel] [PULL 01/19] i2c: Split smbus into parts, (continued)
- [Qemu-devel] [PULL 01/19] i2c: Split smbus into parts, minyard, 2019/02/28
- [Qemu-devel] [PULL 17/19] i2c:smbus_eeprom: Add vmstate handling to the smbus eeprom, minyard, 2019/02/28
- [Qemu-devel] [PULL 04/19] i2c: Don't check return value from i2c_recv(), minyard, 2019/02/28
- [Qemu-devel] [PULL 11/19] i2c:pm_smbus: Fix pm_smbus handling of I2C block read, minyard, 2019/02/28
- [Qemu-devel] [PULL 10/19] boards.h: Ignore migration for SMBus devices on older machines, minyard, 2019/02/28
- [Qemu-devel] [PULL 09/19] i2c:smbus: Make white space in switch statements consistent, minyard, 2019/02/28
- [Qemu-devel] [PULL 16/19] i2c:smbus_eeprom: Add a size constant for the smbus_eeprom size, minyard, 2019/02/28
- [Qemu-devel] [PULL 05/19] i2c:smbus: Correct the working of quick commands, minyard, 2019/02/28
- [Qemu-devel] [PULL 07/19] i2c:smbus: Simplify read handling, minyard, 2019/02/28
- [Qemu-devel] [PULL 12/19] migration: Add a VMSTATE_BOOL_TEST() macro, minyard, 2019/02/28
- [Qemu-devel] [PULL 19/19] i2c: Verify that the count passed in to smbus_eeprom_init() is valid,
minyard <=
- [Qemu-devel] [PULL 14/19] i2c:smbus_slave: Add an SMBus vmstate structure, minyard, 2019/02/28
- [Qemu-devel] [PULL 08/19] i2c:smbus_eeprom: Get rid of the quick command, minyard, 2019/02/28
- [Qemu-devel] [PULL 13/19] i2c:pm_smbus: Fix state transfer, minyard, 2019/02/28