diff options
| author | Fam Zheng | 2018-07-12 04:54:20 +0200 |
|---|---|---|
| committer | Fam Zheng | 2018-08-15 04:12:35 +0200 |
| commit | 9582f357bb6f6573c9a452743d8f3ab41ba2e3fa (patch) | |
| tree | d6055671feca217b0f24f9b7503a1db8710d643f /tests/qapi-schema/nested-struct-data.json | |
| parent | tests/vm: Add flex and bison to the vm image (diff) | |
| download | qemu-9582f357bb6f6573c9a452743d8f3ab41ba2e3fa.tar.gz qemu-9582f357bb6f6573c9a452743d8f3ab41ba2e3fa.tar.xz qemu-9582f357bb6f6573c9a452743d8f3ab41ba2e3fa.zip | |
nvme: Fix nvme_init error handling
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: qemu-stable@nongnu.org
Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20180712025420.4932-1-famz@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Diffstat (limited to 'tests/qapi-schema/nested-struct-data.json')
0 files changed, 0 insertions, 0 deletions