[v3] qcow2: don't leave partially initialized file on image creation

[PATCH v3 0/2] qcow2: don't leave partially initialized file on image creation

Maxim Levitsky posted 2 patches 3 years, 4 months ago

Diff against v2 v4 v5 v6
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201208142159.492489-1-mlevitsk@redhat.com

Maintainers: Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>

There is a newer version of this series

block/crypto.c |  2 ++
block/qcow2.c  | 13 +++++++++++++
2 files changed, 15 insertions(+)

Expand all Fold all

[PATCH v3 0/2] qcow2: don't leave partially initialized file on image creation

Posted by Maxim Levitsky 3 years, 4 months ago

Use the bdrv_co_delete_file interface to delete the underlying
file if qcow2 initialization fails (e.g due to bad encryption secret)

This makes the qcow2 driver behave the same way as the luks driver behaves.

Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1845353

V3: addressed review feedback and reworked commit messages

Best regards,
	Maxim Levitsky

Maxim Levitsky (2):
  crypto: luks: Fix tiny memory leak
  block: qcow2: remove the created file on initialization error

 block/crypto.c |  2 ++
 block/qcow2.c  | 13 +++++++++++++
 2 files changed, 15 insertions(+)

-- 
2.26.2