Re: [Qemu-devel] [PATCH v2] vhost: Fix use-after-free in vhost_log_put()

2017-06-30 Thread Marc-André Lureau
On Fri, Jun 30, 2017 at 2:03 PM Jia-Shiun Li wrote: > In commit 9e0bc24f dev->log_size was reset to zero too early before > syncing vhost log. It causes syncing to be skipped. > > Use local variable to keep its value before resetting. > > Signed-off-by: Jia-Shiun Li > Reviewed-by: Marc-André L

[Qemu-devel] [PATCH v2] vhost: Fix use-after-free in vhost_log_put()

2017-06-30 Thread Jia-Shiun Li
In commit 9e0bc24f dev->log_size was reset to zero too early before syncing vhost log. It causes syncing to be skipped. Use local variable to keep its value before resetting. Signed-off-by: Jia-Shiun Li --- v1 -> v2: * Use local variable to keep value of dev->log_size. --- hw/virtio/vhost.c