[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-block] [PATCH v2] nvme: Fix nvme_init error hand
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-stable] [Qemu-block] [PATCH v2] nvme: Fix nvme_init error handling |
Date: |
Wed, 18 Jul 2018 14:19:51 +0100 |
User-agent: |
Mutt/1.10.0 (2018-05-17) |
On Thu, Jul 12, 2018 at 10:54:20AM +0800, Fam Zheng wrote:
> It is wrong to leave this field as 1, as nvme_close() called in the
> error handling code in nvme_file_open() will use it and try to free
> s->queues again.
>
> Another problem is the cleaning ups are duplicated between the fail*
> labels of nvme_init() and nvme_file_open(), which calls nvme_close().
>
> A third problem is nvme_close() misses g_free() and
> event_notifier_cleanup().
>
> Fix all of them.
>
> Cc: address@hidden
> Signed-off-by: Fam Zheng <address@hidden>
>
> ---
>
> v2: Adopt the suggested fix by Kevin.
> ---
> block/nvme.c | 37 ++++++++++++-------------------------
> 1 file changed, 12 insertions(+), 25 deletions(-)
Reviewed-by: Stefan Hajnoczi <address@hidden>
signature.asc
Description: PGP signature