Re: [Qemu-devel] [PATCH v2 1/2] migration: don't close a file descriptor while it can be in use

2017-04-21 Thread Juan Quintela
Laurent Vivier wrote: > If we close the QEMUFile descriptor in process_incoming_migration_co() > while it has been stopped by an error, the postcopy_ram_listen_thread() > can try to continue to use it. And as the memory has been freed > it is working with an invalid pointer and crashes. > > Fix th

Re: [Qemu-devel] [PATCH v2 1/2] migration: don't close a file descriptor while it can be in use

2017-04-20 Thread Dr. David Alan Gilbert
* Laurent Vivier (lviv...@redhat.com) wrote: > If we close the QEMUFile descriptor in process_incoming_migration_co() > while it has been stopped by an error, the postcopy_ram_listen_thread() > can try to continue to use it. And as the memory has been freed > it is working with an invalid pointer a

[Qemu-devel] [PATCH v2 1/2] migration: don't close a file descriptor while it can be in use

2017-04-12 Thread Laurent Vivier
If we close the QEMUFile descriptor in process_incoming_migration_co() while it has been stopped by an error, the postcopy_ram_listen_thread() can try to continue to use it. And as the memory has been freed it is working with an invalid pointer and crashes. Fix this by releasing the memory after h