[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] migration: don't close a file descriptor
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] migration: don't close a file descriptor while it can be in use |
Date: |
Fri, 21 Apr 2017 11:19:14 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Laurent Vivier <address@hidden> 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 this by releasing the memory after having managed the error
> case (which, in fact, calls exit())
>
> Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>