diff options
author | Kevin Wolf <kwolf@redhat.com> | 2011-02-08 18:12:35 +0100 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2011-02-10 13:23:44 +0100 |
commit | 3ab4c7e92d39d40e6dc0bdb1c2320889543691cb (patch) | |
tree | d7ab94f5e1a60bcaebc79d993ce888e55f16cf16 /block/qcow2.c | |
parent | e0d9c6f93729c9bfc98fcafcd73098bb8e131aeb (diff) | |
download | hqemu-3ab4c7e92d39d40e6dc0bdb1c2320889543691cb.zip hqemu-3ab4c7e92d39d40e6dc0bdb1c2320889543691cb.tar.gz |
qcow2: Fix error handling for immediate backing file read failure
Requests could return success even though they failed when bdrv_aio_readv
returned NULL for a backing file read.
Reported-by: Chunqiang Tang <ctang@us.ibm.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/qcow2.c')
-rw-r--r-- | block/qcow2.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/block/qcow2.c b/block/qcow2.c index 28338bf..647c2a4 100644 --- a/block/qcow2.c +++ b/block/qcow2.c @@ -479,8 +479,10 @@ static void qcow2_aio_read_cb(void *opaque, int ret) BLKDBG_EVENT(bs->file, BLKDBG_READ_BACKING_AIO); acb->hd_aiocb = bdrv_aio_readv(bs->backing_hd, acb->sector_num, &acb->hd_qiov, n1, qcow2_aio_read_cb, acb); - if (acb->hd_aiocb == NULL) + if (acb->hd_aiocb == NULL) { + ret = -EIO; goto done; + } } else { ret = qcow2_schedule_bh(qcow2_aio_read_bh, acb); if (ret < 0) |