[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH 07/23] hw/pci/pcie_aer.c: fix buff
From: |
Peter Maydell |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH 07/23] hw/pci/pcie_aer.c: fix buffer overruns on invalid state load |
Date: |
Tue, 3 Dec 2013 20:59:52 +0000 |
On 3 December 2013 20:41, Michael S. Tsirkin <address@hidden> wrote:
> On Tue, Dec 03, 2013 at 06:30:46PM +0000, Peter Maydell wrote:
>> On 3 December 2013 16:28, Michael S. Tsirkin <address@hidden> wrote:
>> > const VMStateDescription vmstate_pcie_aer_log = {
>> > .name = "PCIE_AER_ERROR_LOG",
>> > .version_id = 1,
>> > .minimum_version_id = 1,
>> > .minimum_version_id_old = 1,
>> > + .post_load = pcie_aer_state_post_load,
>> > .fields = (VMStateField[]) {
>> > VMSTATE_UINT16(log_num, PCIEAERLog),
>> > - VMSTATE_UINT16(log_max, PCIEAERLog),
>> > - VMSTATE_STRUCT_VARRAY_POINTER_UINT16(log, PCIEAERLog, log_num,
>> > + VMSTATE_UINT16_EQUAL(log_max, PCIEAERLog),
>> > + VMSTATE_STRUCT_VARRAY_POINTER_UINT16(log, PCIEAERLog, log_max,
>> > vmstate_pcie_aer_err, PCIEAERErr),
>> > VMSTATE_END_OF_LIST()
>> > }
>>
>> Isn't this a migration compability break?
>
> How is it a break?
If a QEMU with this patch sends data to a QEMU without it, then the
receiving end will think it should expect log_num array entries but the
sending end is going to send log_max of them. Conversely, an old->new
migration is going to send fewer array entries than the destination
expects. Or have I misinterpreted how the VARRAY entries work?
thanks
-- PMM
- Re: [Qemu-stable] [Qemu-devel] [PATCH 01/23] virtio-net: fix buffer overflow on invalid state load, (continued)
- [Qemu-stable] [PATCH 02/23] virtio-net: out-of-bounds buffer write on load, Michael S. Tsirkin, 2013/12/03
- [Qemu-stable] [PATCH 03/23] virtio-net: out-of-bounds buffer write on invalid state load, Michael S. Tsirkin, 2013/12/03
- [Qemu-stable] [PATCH 05/23] ahci: fix buffer overrun on invalid state load, Michael S. Tsirkin, 2013/12/03
- [Qemu-stable] [PATCH 06/23] hpet: fix buffer overrun on invalid state load, Michael S. Tsirkin, 2013/12/03
- [Qemu-stable] [PATCH 07/23] hw/pci/pcie_aer.c: fix buffer overruns on invalid state load, Michael S. Tsirkin, 2013/12/03
[Qemu-stable] [PATCH 09/23] target-arm/machine.c: fix buffer overflow on invalid state load, Michael S. Tsirkin, 2013/12/03
[Qemu-stable] [PATCH 10/23] stellaris_enet: avoid buffer overrun on incoming migration, Michael S. Tsirkin, 2013/12/03
[Qemu-stable] [PATCH 12/23] stellaris_enet: avoid buffer orerrun on incoming migration (part 3), Michael S. Tsirkin, 2013/12/03
[Qemu-stable] [PATCH 13/23] virtio: avoid buffer overrun on incoming migration, Michael S. Tsirkin, 2013/12/03