[v7] delete created files when block_crypto_co_create_opts_luks fails

[Qemu-devel] [PATCH v7 0/4] delete created files when block_crypto_co_create_opts_luks fails

Daniel Henrique Barboza posted 4 patches 6 years, 2 months ago

Diff against v3 v4 v5 v6 v8 v9
Download series mbox

Test docker-clang@ubuntu passed

Test FreeBSD passed

Test checkpatch passed

Test docker-mingw@fedora passed

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

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

There is a newer version of this series

block.c                    | 73 ++++++++++++++++++++++++++++++++++++++
block/crypto.c             | 22 ++++++++++++
block/file-posix.c         | 28 +++++++++++++++
include/block/block.h      |  1 +
include/block/block_int.h  |  6 ++++
tests/qemu-iotests/259     | 67 ++++++++++++++++++++++++++++++++++
tests/qemu-iotests/259.out | 11 ++++++
tests/qemu-iotests/group   |  1 +
8 files changed, 209 insertions(+)
create mode 100755 tests/qemu-iotests/259
create mode 100644 tests/qemu-iotests/259.out

Expand all Fold all

[Qemu-devel] [PATCH v7 0/4] delete created files when block_crypto_co_create_opts_luks fails

Posted by Daniel Henrique Barboza 6 years, 2 months ago

Changes from previous version 6 [1]:

- bdrv_delete_file() now uses BlockDriverState as a parameter rather
than a filename string.

[1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg00139.html

Daniel Henrique Barboza (4):
  block: introducing 'bdrv_co_delete_file' interface
  block.c: adding bdrv_delete_file
  crypto.c: cleanup created file when block_crypto_co_create_opts_luks
    fails
  qemu-iotests: adding LUKS cleanup for non-UTF8 secret error

 block.c                    | 73 ++++++++++++++++++++++++++++++++++++++
 block/crypto.c             | 22 ++++++++++++
 block/file-posix.c         | 28 +++++++++++++++
 include/block/block.h      |  1 +
 include/block/block_int.h  |  6 ++++
 tests/qemu-iotests/259     | 67 ++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/259.out | 11 ++++++
 tests/qemu-iotests/group   |  1 +
 8 files changed, 209 insertions(+)
 create mode 100755 tests/qemu-iotests/259
 create mode 100644 tests/qemu-iotests/259.out

-- 
2.21.0

Re: [PATCH v7 0/4] delete created files when block_crypto_co_create_opts_luks fails

Posted by Daniel Henrique Barboza 6 years ago

Ping

On 9/3/19 10:57 AM, Daniel Henrique Barboza wrote:
> Changes from previous version 6 [1]:
> 
> - bdrv_delete_file() now uses BlockDriverState as a parameter rather
> than a filename string.
> 
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg00139.html
> 
> Daniel Henrique Barboza (4):
>    block: introducing 'bdrv_co_delete_file' interface
>    block.c: adding bdrv_delete_file
>    crypto.c: cleanup created file when block_crypto_co_create_opts_luks
>      fails
>    qemu-iotests: adding LUKS cleanup for non-UTF8 secret error
> 
>   block.c                    | 73 ++++++++++++++++++++++++++++++++++++++
>   block/crypto.c             | 22 ++++++++++++
>   block/file-posix.c         | 28 +++++++++++++++
>   include/block/block.h      |  1 +
>   include/block/block_int.h  |  6 ++++
>   tests/qemu-iotests/259     | 67 ++++++++++++++++++++++++++++++++++
>   tests/qemu-iotests/259.out | 11 ++++++
>   tests/qemu-iotests/group   |  1 +
>   8 files changed, 209 insertions(+)
>   create mode 100755 tests/qemu-iotests/259
>   create mode 100644 tests/qemu-iotests/259.out
>

Re: [PATCH v7 0/4] delete created files when block_crypto_co_create_opts_luks fails

Posted by Daniel Henrique Barboza 6 years ago

Ping

On 9/3/19 10:57 AM, Daniel Henrique Barboza wrote:
> Changes from previous version 6 [1]:
>
> - bdrv_delete_file() now uses BlockDriverState as a parameter rather
> than a filename string.
>
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg00139.html
>
> Daniel Henrique Barboza (4):
>    block: introducing 'bdrv_co_delete_file' interface
>    block.c: adding bdrv_delete_file
>    crypto.c: cleanup created file when block_crypto_co_create_opts_luks
>      fails
>    qemu-iotests: adding LUKS cleanup for non-UTF8 secret error
>
>   block.c                    | 73 ++++++++++++++++++++++++++++++++++++++
>   block/crypto.c             | 22 ++++++++++++
>   block/file-posix.c         | 28 +++++++++++++++
>   include/block/block.h      |  1 +
>   include/block/block_int.h  |  6 ++++
>   tests/qemu-iotests/259     | 67 ++++++++++++++++++++++++++++++++++
>   tests/qemu-iotests/259.out | 11 ++++++
>   tests/qemu-iotests/group   |  1 +
>   8 files changed, 209 insertions(+)
>   create mode 100755 tests/qemu-iotests/259
>   create mode 100644 tests/qemu-iotests/259.out
>