Re: [Qemu-devel] [PATCH v2] usb: fix up post load checks

2014-05-14 Thread Juan Quintela
"Michael S. Tsirkin" wrote: > Correct post load checks: > 1. dev->setup_len == sizeof(dev->data_buf) > seems fine, no need to fail migration > 2. When state is DATA, passing index > len >will cause memcpy with negative length, >resulting in heap overflow > > First of the issues was rep

[Qemu-devel] [PATCH v2] usb: fix up post load checks

2014-05-13 Thread Michael S. Tsirkin
Correct post load checks: 1. dev->setup_len == sizeof(dev->data_buf) seems fine, no need to fail migration 2. When state is DATA, passing index > len will cause memcpy with negative length, resulting in heap overflow First of the issues was reported by dgilbert. Reported-by: "Dr. David