diff options
| author | Kevin Wolf | 2017-03-03 16:54:21 +0100 |
|---|---|---|
| committer | Kevin Wolf | 2017-03-07 14:53:28 +0100 |
| commit | b247767aacdc2cfba94050c901aec268be94cc2d (patch) | |
| tree | 6584385431aaa5319ad3e73da624c5d0a08a89d1 | |
| parent | Merge remote-tracking branch 'remotes/xtensa/tags/20170306-xtensa' into staging (diff) | |
| download | qemu-b247767aacdc2cfba94050c901aec268be94cc2d.tar.gz qemu-b247767aacdc2cfba94050c901aec268be94cc2d.tar.xz qemu-b247767aacdc2cfba94050c901aec268be94cc2d.zip | |
commit: Fix error handling
Apparently some kind of mismerge happened in commit 8dfba279, which
broke the error handling without any real reason by removing the
assignment of the return value to ret in a blk_insert_bs() call.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
| -rw-r--r-- | block/commit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/commit.c b/block/commit.c index 22a0a4db98..e57c1cffa2 100644 --- a/block/commit.c +++ b/block/commit.c @@ -364,7 +364,7 @@ void commit_start(const char *job_id, BlockDriverState *bs, /* Required permissions are already taken with block_job_add_bdrv() */ s->top = blk_new(0, BLK_PERM_ALL); - blk_insert_bs(s->top, top, errp); + ret = blk_insert_bs(s->top, top, errp); if (ret < 0) { goto fail; } |