| commit | cfc87e00c22ab4ea0262c9771c803ed03d754001 | |
| author | Peter Maydell <peter.maydell@linaro.org> | |
| Sun, 9 Jul 2017 21:07:17 +0000 (9 22:07 +0100) | ||
| committer | Kevin Wolf <kwolf@redhat.com> | |
| Tue, 18 Jul 2017 13:14:35 +0000 (18 15:14 +0200) | ||
| tree | 66e44f75ce2db3579d9ff5704801e3ad54974224 | treesnapshot (tar.gz zip) |
| parent | 9877860e7bd1e26ee70ab9bb5ebc34c92bf23bf5 | commitdiff |
block/vpc.c: Handle write failures in get_image_offset()
Coverity (CID 1355236) points out that get_image_offset() doesn't check that
it actually succeeded in writing the updated block bitmap to the file.
Check the error return from bdrv_pwrite_sync() and propagate an error
response back up to the function which calls get_image_offset() for
a write so that it can return the error to its caller.
get_sector_offset() is only used for reads, but we move it to the
same API for consistency.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Coverity (CID 1355236) points out that get_image_offset() doesn't check that
it actually succeeded in writing the updated block bitmap to the file.
Check the error return from bdrv_pwrite_sync() and propagate an error
response back up to the function which calls get_image_offset() for
a write so that it can return the error to its caller.
get_sector_offset() is only used for reads, but we move it to the
same API for consistency.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
| block/vpc.c | diffblobblamehistory |