[v1] block: More robust error handling around perm check

[Qemu-devel] [PATCH 0/2] block: More robust error handling around perm check

Fam Zheng posted 2 patches 7 years, 1 month ago

Diff against v2
Download series mbox

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

Test checkpatch passed

Test docker passed

There is a newer version of this series

block/block-backend.c |  6 +++++-
block/commit.c        | 12 ++++++++++--
2 files changed, 15 insertions(+), 3 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] block: More robust error handling around perm check

Posted by Fam Zheng 7 years, 1 month ago

These are a couple cases added with the op blocker changes, where errp is
available but we still use error_abort. Fix them for obvious reasons.

Fam Zheng (2):
  block: Don't use error_abort in blk_new_open
  commit: Don't use error_abort in commit_start

 block/block-backend.c |  6 +++++-
 block/commit.c        | 12 ++++++++++--
 2 files changed, 15 insertions(+), 3 deletions(-)

-- 
2.9.3

Re: [Qemu-devel] [PATCH for-2.9 0/2] block: More robust error handling around perm check

Posted by Eric Blake 7 years, 1 month ago

On 03/03/2017 07:38 AM, Fam Zheng wrote:
> These are a couple cases added with the op blocker changes, where errp is
> available but we still use error_abort. Fix them for obvious reasons.
> 
> Fam Zheng (2):
>   block: Don't use error_abort in blk_new_open
>   commit: Don't use error_abort in commit_start

Reviewed-by: Eric Blake <eblake@redhat.com>

> 
>  block/block-backend.c |  6 +++++-
>  block/commit.c        | 12 ++++++++++--
>  2 files changed, 15 insertions(+), 3 deletions(-)
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org